决策:#7391 在 @object-ui/react 唯一入口上发布了 5 个符号,而裁定 a′ 写的是「the constant is internal」——就地追认、迁 core、还是收回? #7508

Description

@os-project-manager

一句话

#7210 的裁定 a′ 逐字写着 「Clause-② no (no contract change; the constant is internal and the footnote is renderer copy)」,而落地的 PR #7391packages/react/src/index.ts发布了五个符号。裁定的前提没有成立,该发布面今天已经在 main 上。⛔ 本席不裁——这是对已发布 API 的处置,且与维护者裁定的字面直接相抵。

现在做便宜,以后不便宜:审计实测,除四个渲染器与测试外今天没有任何消费者 import 这五个符号中的任何一个。这个窗口一旦有下游采纳就关上了。

发布了什么

packages/react/src/index.ts:100-113:

export{NON_GRID_ROW_CEILING,NON_GRID_ROW_CEILING_TOP,applyNonGridRowCeiling,NonGridRowCeilingNote,}from'./utils/nonGridRowCeiling.js';exporttype{NonGridCeilingResult}from'./utils/nonGridRowCeiling.js';

为什么会走到这里(责任在派发席,记在明处)

不是执行车道擅自扩面。那一轮本席自己下的 barrel 围栏把出路堵死了:#7210 评论 5511301369 声明 core#7349types+components#7182 当轮所有,「需要就停下回报」。四个渲染器共用一个常量,唯一没被围住的包是 react,而 react没有 subpath export ⇒ 「把常量放进 react」在物理上只能等于「发布它」。

⚠️ 车道当时应当停下回报(围栏原话如此),而不是铸出 API;但围栏的写法没有点明「若唯一剩下的家是已发布入口,停」。两边各承一半,前一半是本席的。

index.ts:100-104 的注释还把这段历史写成了一个实测为假的技术理由——「the only package all four already depend on」;四个插件的 package.json 各自同时依赖 corecomponentstypes。该注释的订正已单独立卡 #7507(仅改注释,⛔ 不动符号)。

逐符号的可支撑性(审计读数)

符号读数
applyNonGridRowCeiling / NonGridCeilingResult可支撑,可加性演进(可选 options 参数、可加字段)。⚠️total 非来源可信:服务端不报 totaldata-objectstackrecords.length 顶替(packages/data-objectstack/src/index.ts:3624-3646),这类服务端上脚注会读成「first 2000 of 2001」。ObjectStack 服务端报 total,故属边界注记。
NON_GRID_ROW_CEILING可支撑,但家不对:applyNonGridRowCeilingcore 自己的 extractRecords 的包装,四个插件本来就从 core import 它。
NON_GRID_ROW_CEILING_TOP实现细节被发布。它存在只因机制被劈成两半($top 在调用点、切片在 helper)。任何未来的截断探测策略(hasMore、可信 total)都会让它退化成残件。
NonGridRowCeilingNote形状是陷阱:收 drawn/total/truncated 三个散 prop,而每个调用点都硬写 drawn={NON_GRID_ROW_CEILING}。组件从不见到它本该渲染的 NonGridCeilingResult ⇒ 不自洽的脚注是可表达的。(组件出现在该入口本身有先例:SchemaRendererI18nProviderElementDataSourceGate。异议在形状,不在类别。)

选项

A —— 迁 @object-ui/core,react 保留再导出(审计的建议)。NON_GRID_ROW_CEILINGapplyNonGridRowCeilingNonGridCeilingResult 移到 core(extractRecords 旁边),react 继续 export … from '@object-ui/core'——react 早已这样再导出下层符号(export { I18nProvider, … } from '@object-ui/i18n'),先例在。零破坏。 另加性收 _TOP(导出一个 nonGridRowCeilingQuery(): { $top },调用点再不手写 +1)与重塑 Note(收 result: NonGridCeilingResult,drawnresult.rows.length 推得,旧散 prop 保留为 deprecated 路径)。
⇒ 修好了家与形状,但五个符号仍是公开 API,裁定 a′ 的「internal」并未回复。

B —— 回到裁定的字面:收回发布面。 五个符号全部下架,常量落在一个四个插件都能吃到而不经已发布入口的位置。⚠️ 本席读下来这可能做不到:每个包都只有唯一入口,react 无 subpath export ⇒ B 实际要求先给某个包开 subpath export(或新开一个内部包),那本身是更大的一步。若维护者要 B,请一并裁明这一步走哪条。

C —— 就地追认。 承认裁定 a′ 的「internal」前提在本仓的包形状下不可达,把五个符号作为公开 API 追认,只做 #7507 的注释订正与 _TOP 的文档降级(标注为派生/测试用)。
⇒ 最省,但把一个「因排期围栏而选中的家」永久冻进公开 API。

本席倾向:A

理由:(a) 真实需求——core 是四个插件本来就依赖并从中取 extractRecords 的包,家对了代码才读得顺;(b) 长期健全——_TOPNote 的两处加性修整把「不自洽脚注可表达」和「调用点手写 +1」两个真陷阱都关掉,且都零破坏;(c) AI 不易做错——nonGridRowCeilingQuery()+1 只有一处;(d) 创业期尺度——今天零外部消费者,A 是一次性的便宜迁移,B 要先开 subpath export(实现优先、无人拉动),C 则把已知错的家冻住。

⛔ 但 A 不回复裁定 a′ 的字面前提,所以它需要维护者点头而不是本席自裁。

派工状态

⛔ 本卡不可派工,needs-user-decision。不带 pm:queue 标签——本卡在裁定前不进队列,这是 #6853 点名的缺陷(卡片自己说不可派工,而标签仍在招手)。

不受本卡阻塞、已单独立卡并本轮派工的:#7507(pin 不判定上限、docblock 判别机制写错、changeset 档位与文案、calendar 陈旧 ceiling、假的依赖注释)。审计全文逐字见 #7391 评论 5527977938。

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpm:queuepriority:p2

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

      , 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
       blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
      }
      } catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
      })();
      (function(){
      try {
      var __m = "github.com";
      var __re = new RegExp('^' + "github\\.com" + '
      
      Skip to content

      决策:#7391 在 @object-ui/react 唯一入口上发布了 5 个符号,而裁定 a′ 写的是「the constant is internal」——就地追认、迁 core、还是收回? #7508

      Description

      @os-project-manager

      一句话

      #7210 的裁定 a′ 逐字写着 「Clause-② no (no contract change; the constant is internal and the footnote is renderer copy)」,而落地的 PR #7391packages/react/src/index.ts发布了五个符号。裁定的前提没有成立,该发布面今天已经在 main 上。⛔ 本席不裁——这是对已发布 API 的处置,且与维护者裁定的字面直接相抵。

      现在做便宜,以后不便宜:审计实测,除四个渲染器与测试外今天没有任何消费者 import 这五个符号中的任何一个。这个窗口一旦有下游采纳就关上了。

      发布了什么

      packages/react/src/index.ts:100-113:

      export{NON_GRID_ROW_CEILING,NON_GRID_ROW_CEILING_TOP,applyNonGridRowCeiling,NonGridRowCeilingNote,}from'./utils/nonGridRowCeiling.js';exporttype{NonGridCeilingResult}from'./utils/nonGridRowCeiling.js';

      为什么会走到这里(责任在派发席,记在明处)

      不是执行车道擅自扩面。那一轮本席自己下的 barrel 围栏把出路堵死了:#7210 评论 5511301369 声明 core#7349types+components#7182 当轮所有,「需要就停下回报」。四个渲染器共用一个常量,唯一没被围住的包是 react,而 react没有 subpath export ⇒ 「把常量放进 react」在物理上只能等于「发布它」。

      ⚠️ 车道当时应当停下回报(围栏原话如此),而不是铸出 API;但围栏的写法没有点明「若唯一剩下的家是已发布入口,停」。两边各承一半,前一半是本席的。

      index.ts:100-104 的注释还把这段历史写成了一个实测为假的技术理由——「the only package all four already depend on」;四个插件的 package.json 各自同时依赖 corecomponentstypes。该注释的订正已单独立卡 #7507(仅改注释,⛔ 不动符号)。

      逐符号的可支撑性(审计读数)

      符号读数
      applyNonGridRowCeiling / NonGridCeilingResult可支撑,可加性演进(可选 options 参数、可加字段)。⚠️total 非来源可信:服务端不报 totaldata-objectstackrecords.length 顶替(packages/data-objectstack/src/index.ts:3624-3646),这类服务端上脚注会读成「first 2000 of 2001」。ObjectStack 服务端报 total,故属边界注记。
      NON_GRID_ROW_CEILING可支撑,但家不对:applyNonGridRowCeilingcore 自己的 extractRecords 的包装,四个插件本来就从 core import 它。
      NON_GRID_ROW_CEILING_TOP实现细节被发布。它存在只因机制被劈成两半($top 在调用点、切片在 helper)。任何未来的截断探测策略(hasMore、可信 total)都会让它退化成残件。
      NonGridRowCeilingNote形状是陷阱:收 drawn/total/truncated 三个散 prop,而每个调用点都硬写 drawn={NON_GRID_ROW_CEILING}。组件从不见到它本该渲染的 NonGridCeilingResult ⇒ 不自洽的脚注是可表达的。(组件出现在该入口本身有先例:SchemaRendererI18nProviderElementDataSourceGate。异议在形状,不在类别。)

      选项

      A —— 迁 @object-ui/core,react 保留再导出(审计的建议)。NON_GRID_ROW_CEILINGapplyNonGridRowCeilingNonGridCeilingResult 移到 core(extractRecords 旁边),react 继续 export … from '@object-ui/core'——react 早已这样再导出下层符号(export { I18nProvider, … } from '@object-ui/i18n'),先例在。零破坏。 另加性收 _TOP(导出一个 nonGridRowCeilingQuery(): { $top },调用点再不手写 +1)与重塑 Note(收 result: NonGridCeilingResult,drawnresult.rows.length 推得,旧散 prop 保留为 deprecated 路径)。
      ⇒ 修好了家与形状,但五个符号仍是公开 API,裁定 a′ 的「internal」并未回复。

      B —— 回到裁定的字面:收回发布面。 五个符号全部下架,常量落在一个四个插件都能吃到而不经已发布入口的位置。⚠️ 本席读下来这可能做不到:每个包都只有唯一入口,react 无 subpath export ⇒ B 实际要求先给某个包开 subpath export(或新开一个内部包),那本身是更大的一步。若维护者要 B,请一并裁明这一步走哪条。

      C —— 就地追认。 承认裁定 a′ 的「internal」前提在本仓的包形状下不可达,把五个符号作为公开 API 追认,只做 #7507 的注释订正与 _TOP 的文档降级(标注为派生/测试用)。
      ⇒ 最省,但把一个「因排期围栏而选中的家」永久冻进公开 API。

      本席倾向:A

      理由:(a) 真实需求——core 是四个插件本来就依赖并从中取 extractRecords 的包,家对了代码才读得顺;(b) 长期健全——_TOPNote 的两处加性修整把「不自洽脚注可表达」和「调用点手写 +1」两个真陷阱都关掉,且都零破坏;(c) AI 不易做错——nonGridRowCeilingQuery()+1 只有一处;(d) 创业期尺度——今天零外部消费者,A 是一次性的便宜迁移,B 要先开 subpath export(实现优先、无人拉动),C 则把已知错的家冻住。

      ⛔ 但 A 不回复裁定 a′ 的字面前提,所以它需要维护者点头而不是本席自裁。

      派工状态

      ⛔ 本卡不可派工,needs-user-decision。不带 pm:queue 标签——本卡在裁定前不进队列,这是 #6853 点名的缺陷(卡片自己说不可派工,而标签仍在招手)。

      不受本卡阻塞、已单独立卡并本轮派工的:#7507(pin 不判定上限、docblock 判别机制写错、changeset 档位与文案、calendar 陈旧 ceiling、假的依赖注释)。审计全文逐字见 #7391 评论 5527977938。

      Activity

      Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

      Metadata

      Metadata

      Assignees

      No one assigned

        Labels

        domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpm:queuepriority:p2

        Type

        No type

        Projects

        No projects

          Milestone

          No milestone

          Relationships

          None yet

          Development

          No branches or pull requests

          Issue actions

          , 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
          Skip to content

          决策:#7391 在 @object-ui/react 唯一入口上发布了 5 个符号,而裁定 a′ 写的是「the constant is internal」——就地追认、迁 core、还是收回? #7508

          Description

          @os-project-manager

          一句话

          #7210 的裁定 a′ 逐字写着 「Clause-② no (no contract change; the constant is internal and the footnote is renderer copy)」,而落地的 PR #7391packages/react/src/index.ts发布了五个符号。裁定的前提没有成立,该发布面今天已经在 main 上。⛔ 本席不裁——这是对已发布 API 的处置,且与维护者裁定的字面直接相抵。

          现在做便宜,以后不便宜:审计实测,除四个渲染器与测试外今天没有任何消费者 import 这五个符号中的任何一个。这个窗口一旦有下游采纳就关上了。

          发布了什么

          packages/react/src/index.ts:100-113:

          export{NON_GRID_ROW_CEILING,NON_GRID_ROW_CEILING_TOP,applyNonGridRowCeiling,NonGridRowCeilingNote,}from'./utils/nonGridRowCeiling.js';exporttype{NonGridCeilingResult}from'./utils/nonGridRowCeiling.js';

          为什么会走到这里(责任在派发席,记在明处)

          不是执行车道擅自扩面。那一轮本席自己下的 barrel 围栏把出路堵死了:#7210 评论 5511301369 声明 core#7349types+components#7182 当轮所有,「需要就停下回报」。四个渲染器共用一个常量,唯一没被围住的包是 react,而 react没有 subpath export ⇒ 「把常量放进 react」在物理上只能等于「发布它」。

          ⚠️ 车道当时应当停下回报(围栏原话如此),而不是铸出 API;但围栏的写法没有点明「若唯一剩下的家是已发布入口,停」。两边各承一半,前一半是本席的。

          index.ts:100-104 的注释还把这段历史写成了一个实测为假的技术理由——「the only package all four already depend on」;四个插件的 package.json 各自同时依赖 corecomponentstypes。该注释的订正已单独立卡 #7507(仅改注释,⛔ 不动符号)。

          逐符号的可支撑性(审计读数)

          符号读数
          applyNonGridRowCeiling / NonGridCeilingResult可支撑,可加性演进(可选 options 参数、可加字段)。⚠️total 非来源可信:服务端不报 totaldata-objectstackrecords.length 顶替(packages/data-objectstack/src/index.ts:3624-3646),这类服务端上脚注会读成「first 2000 of 2001」。ObjectStack 服务端报 total,故属边界注记。
          NON_GRID_ROW_CEILING可支撑,但家不对:applyNonGridRowCeilingcore 自己的 extractRecords 的包装,四个插件本来就从 core import 它。
          NON_GRID_ROW_CEILING_TOP实现细节被发布。它存在只因机制被劈成两半($top 在调用点、切片在 helper)。任何未来的截断探测策略(hasMore、可信 total)都会让它退化成残件。
          NonGridRowCeilingNote形状是陷阱:收 drawn/total/truncated 三个散 prop,而每个调用点都硬写 drawn={NON_GRID_ROW_CEILING}。组件从不见到它本该渲染的 NonGridCeilingResult ⇒ 不自洽的脚注是可表达的。(组件出现在该入口本身有先例:SchemaRendererI18nProviderElementDataSourceGate。异议在形状,不在类别。)

          选项

          A —— 迁 @object-ui/core,react 保留再导出(审计的建议)。NON_GRID_ROW_CEILINGapplyNonGridRowCeilingNonGridCeilingResult 移到 core(extractRecords 旁边),react 继续 export … from '@object-ui/core'——react 早已这样再导出下层符号(export { I18nProvider, … } from '@object-ui/i18n'),先例在。零破坏。 另加性收 _TOP(导出一个 nonGridRowCeilingQuery(): { $top },调用点再不手写 +1)与重塑 Note(收 result: NonGridCeilingResult,drawnresult.rows.length 推得,旧散 prop 保留为 deprecated 路径)。
          ⇒ 修好了家与形状,但五个符号仍是公开 API,裁定 a′ 的「internal」并未回复。

          B —— 回到裁定的字面:收回发布面。 五个符号全部下架,常量落在一个四个插件都能吃到而不经已发布入口的位置。⚠️ 本席读下来这可能做不到:每个包都只有唯一入口,react 无 subpath export ⇒ B 实际要求先给某个包开 subpath export(或新开一个内部包),那本身是更大的一步。若维护者要 B,请一并裁明这一步走哪条。

          C —— 就地追认。 承认裁定 a′ 的「internal」前提在本仓的包形状下不可达,把五个符号作为公开 API 追认,只做 #7507 的注释订正与 _TOP 的文档降级(标注为派生/测试用)。
          ⇒ 最省,但把一个「因排期围栏而选中的家」永久冻进公开 API。

          本席倾向:A

          理由:(a) 真实需求——core 是四个插件本来就依赖并从中取 extractRecords 的包,家对了代码才读得顺;(b) 长期健全——_TOPNote 的两处加性修整把「不自洽脚注可表达」和「调用点手写 +1」两个真陷阱都关掉,且都零破坏;(c) AI 不易做错——nonGridRowCeilingQuery()+1 只有一处;(d) 创业期尺度——今天零外部消费者,A 是一次性的便宜迁移,B 要先开 subpath export(实现优先、无人拉动),C 则把已知错的家冻住。

          ⛔ 但 A 不回复裁定 a′ 的字面前提,所以它需要维护者点头而不是本席自裁。

          派工状态

          ⛔ 本卡不可派工,needs-user-decision。不带 pm:queue 标签——本卡在裁定前不进队列,这是 #6853 点名的缺陷(卡片自己说不可派工,而标签仍在招手)。

          不受本卡阻塞、已单独立卡并本轮派工的:#7507(pin 不判定上限、docblock 判别机制写错、changeset 档位与文案、calendar 陈旧 ceiling、假的依赖注释)。审计全文逐字见 #7391 评论 5527977938。

          Activity

          Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

          Metadata

          Metadata

          Assignees

          No one assigned

            Labels

            domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpm:queuepriority:p2

            Type

            No type

            Projects

            No projects

              Milestone

              No milestone

              Relationships

              None yet

              Development

              No branches or pull requests

              Issue actions

              , 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
              Skip to content

              决策:#7391 在 @object-ui/react 唯一入口上发布了 5 个符号,而裁定 a′ 写的是「the constant is internal」——就地追认、迁 core、还是收回? #7508

              Description

              @os-project-manager

              一句话

              #7210 的裁定 a′ 逐字写着 「Clause-② no (no contract change; the constant is internal and the footnote is renderer copy)」,而落地的 PR #7391packages/react/src/index.ts发布了五个符号。裁定的前提没有成立,该发布面今天已经在 main 上。⛔ 本席不裁——这是对已发布 API 的处置,且与维护者裁定的字面直接相抵。

              现在做便宜,以后不便宜:审计实测,除四个渲染器与测试外今天没有任何消费者 import 这五个符号中的任何一个。这个窗口一旦有下游采纳就关上了。

              发布了什么

              packages/react/src/index.ts:100-113:

              export{NON_GRID_ROW_CEILING,NON_GRID_ROW_CEILING_TOP,applyNonGridRowCeiling,NonGridRowCeilingNote,}from'./utils/nonGridRowCeiling.js';exporttype{NonGridCeilingResult}from'./utils/nonGridRowCeiling.js';

              为什么会走到这里(责任在派发席,记在明处)

              不是执行车道擅自扩面。那一轮本席自己下的 barrel 围栏把出路堵死了:#7210 评论 5511301369 声明 core#7349types+components#7182 当轮所有,「需要就停下回报」。四个渲染器共用一个常量,唯一没被围住的包是 react,而 react没有 subpath export ⇒ 「把常量放进 react」在物理上只能等于「发布它」。

              ⚠️ 车道当时应当停下回报(围栏原话如此),而不是铸出 API;但围栏的写法没有点明「若唯一剩下的家是已发布入口,停」。两边各承一半,前一半是本席的。

              index.ts:100-104 的注释还把这段历史写成了一个实测为假的技术理由——「the only package all four already depend on」;四个插件的 package.json 各自同时依赖 corecomponentstypes。该注释的订正已单独立卡 #7507(仅改注释,⛔ 不动符号)。

              逐符号的可支撑性(审计读数)

              符号读数
              applyNonGridRowCeiling / NonGridCeilingResult可支撑,可加性演进(可选 options 参数、可加字段)。⚠️total 非来源可信:服务端不报 totaldata-objectstackrecords.length 顶替(packages/data-objectstack/src/index.ts:3624-3646),这类服务端上脚注会读成「first 2000 of 2001」。ObjectStack 服务端报 total,故属边界注记。
              NON_GRID_ROW_CEILING可支撑,但家不对:applyNonGridRowCeilingcore 自己的 extractRecords 的包装,四个插件本来就从 core import 它。
              NON_GRID_ROW_CEILING_TOP实现细节被发布。它存在只因机制被劈成两半($top 在调用点、切片在 helper)。任何未来的截断探测策略(hasMore、可信 total)都会让它退化成残件。
              NonGridRowCeilingNote形状是陷阱:收 drawn/total/truncated 三个散 prop,而每个调用点都硬写 drawn={NON_GRID_ROW_CEILING}。组件从不见到它本该渲染的 NonGridCeilingResult ⇒ 不自洽的脚注是可表达的。(组件出现在该入口本身有先例:SchemaRendererI18nProviderElementDataSourceGate。异议在形状,不在类别。)

              选项

              A —— 迁 @object-ui/core,react 保留再导出(审计的建议)。NON_GRID_ROW_CEILINGapplyNonGridRowCeilingNonGridCeilingResult 移到 core(extractRecords 旁边),react 继续 export … from '@object-ui/core'——react 早已这样再导出下层符号(export { I18nProvider, … } from '@object-ui/i18n'),先例在。零破坏。 另加性收 _TOP(导出一个 nonGridRowCeilingQuery(): { $top },调用点再不手写 +1)与重塑 Note(收 result: NonGridCeilingResult,drawnresult.rows.length 推得,旧散 prop 保留为 deprecated 路径)。
              ⇒ 修好了家与形状,但五个符号仍是公开 API,裁定 a′ 的「internal」并未回复。

              B —— 回到裁定的字面:收回发布面。 五个符号全部下架,常量落在一个四个插件都能吃到而不经已发布入口的位置。⚠️ 本席读下来这可能做不到:每个包都只有唯一入口,react 无 subpath export ⇒ B 实际要求先给某个包开 subpath export(或新开一个内部包),那本身是更大的一步。若维护者要 B,请一并裁明这一步走哪条。

              C —— 就地追认。 承认裁定 a′ 的「internal」前提在本仓的包形状下不可达,把五个符号作为公开 API 追认,只做 #7507 的注释订正与 _TOP 的文档降级(标注为派生/测试用)。
              ⇒ 最省,但把一个「因排期围栏而选中的家」永久冻进公开 API。

              本席倾向:A

              理由:(a) 真实需求——core 是四个插件本来就依赖并从中取 extractRecords 的包,家对了代码才读得顺;(b) 长期健全——_TOPNote 的两处加性修整把「不自洽脚注可表达」和「调用点手写 +1」两个真陷阱都关掉,且都零破坏;(c) AI 不易做错——nonGridRowCeilingQuery()+1 只有一处;(d) 创业期尺度——今天零外部消费者,A 是一次性的便宜迁移,B 要先开 subpath export(实现优先、无人拉动),C 则把已知错的家冻住。

              ⛔ 但 A 不回复裁定 a′ 的字面前提,所以它需要维护者点头而不是本席自裁。

              派工状态

              ⛔ 本卡不可派工,needs-user-decision。不带 pm:queue 标签——本卡在裁定前不进队列,这是 #6853 点名的缺陷(卡片自己说不可派工,而标签仍在招手)。

              不受本卡阻塞、已单独立卡并本轮派工的:#7507(pin 不判定上限、docblock 判别机制写错、changeset 档位与文案、calendar 陈旧 ceiling、假的依赖注释)。审计全文逐字见 #7391 评论 5527977938。

              Activity

              Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

              Metadata

              Metadata

              Assignees

              No one assigned

                Labels

                domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpm:queuepriority:p2

                Type

                No type

                Projects

                No projects

                  Milestone

                  No milestone

                  Relationships

                  None yet

                  Development

                  No branches or pull requests

                  Issue actions

                  , 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
                  Skip to content

                  决策:#7391 在 @object-ui/react 唯一入口上发布了 5 个符号,而裁定 a′ 写的是「the constant is internal」——就地追认、迁 core、还是收回? #7508

                  Description

                  @os-project-manager

                  一句话

                  #7210 的裁定 a′ 逐字写着 「Clause-② no (no contract change; the constant is internal and the footnote is renderer copy)」,而落地的 PR #7391packages/react/src/index.ts发布了五个符号。裁定的前提没有成立,该发布面今天已经在 main 上。⛔ 本席不裁——这是对已发布 API 的处置,且与维护者裁定的字面直接相抵。

                  现在做便宜,以后不便宜:审计实测,除四个渲染器与测试外今天没有任何消费者 import 这五个符号中的任何一个。这个窗口一旦有下游采纳就关上了。

                  发布了什么

                  packages/react/src/index.ts:100-113:

                  export{NON_GRID_ROW_CEILING,NON_GRID_ROW_CEILING_TOP,applyNonGridRowCeiling,NonGridRowCeilingNote,}from'./utils/nonGridRowCeiling.js';exporttype{NonGridCeilingResult}from'./utils/nonGridRowCeiling.js';

                  为什么会走到这里(责任在派发席,记在明处)

                  不是执行车道擅自扩面。那一轮本席自己下的 barrel 围栏把出路堵死了:#7210 评论 5511301369 声明 core#7349types+components#7182 当轮所有,「需要就停下回报」。四个渲染器共用一个常量,唯一没被围住的包是 react,而 react没有 subpath export ⇒ 「把常量放进 react」在物理上只能等于「发布它」。

                  ⚠️ 车道当时应当停下回报(围栏原话如此),而不是铸出 API;但围栏的写法没有点明「若唯一剩下的家是已发布入口,停」。两边各承一半,前一半是本席的。

                  index.ts:100-104 的注释还把这段历史写成了一个实测为假的技术理由——「the only package all four already depend on」;四个插件的 package.json 各自同时依赖 corecomponentstypes。该注释的订正已单独立卡 #7507(仅改注释,⛔ 不动符号)。

                  逐符号的可支撑性(审计读数)

                  符号读数
                  applyNonGridRowCeiling / NonGridCeilingResult可支撑,可加性演进(可选 options 参数、可加字段)。⚠️total 非来源可信:服务端不报 totaldata-objectstackrecords.length 顶替(packages/data-objectstack/src/index.ts:3624-3646),这类服务端上脚注会读成「first 2000 of 2001」。ObjectStack 服务端报 total,故属边界注记。
                  NON_GRID_ROW_CEILING可支撑,但家不对:applyNonGridRowCeilingcore 自己的 extractRecords 的包装,四个插件本来就从 core import 它。
                  NON_GRID_ROW_CEILING_TOP实现细节被发布。它存在只因机制被劈成两半($top 在调用点、切片在 helper)。任何未来的截断探测策略(hasMore、可信 total)都会让它退化成残件。
                  NonGridRowCeilingNote形状是陷阱:收 drawn/total/truncated 三个散 prop,而每个调用点都硬写 drawn={NON_GRID_ROW_CEILING}。组件从不见到它本该渲染的 NonGridCeilingResult ⇒ 不自洽的脚注是可表达的。(组件出现在该入口本身有先例:SchemaRendererI18nProviderElementDataSourceGate。异议在形状,不在类别。)

                  选项

                  A —— 迁 @object-ui/core,react 保留再导出(审计的建议)。NON_GRID_ROW_CEILINGapplyNonGridRowCeilingNonGridCeilingResult 移到 core(extractRecords 旁边),react 继续 export … from '@object-ui/core'——react 早已这样再导出下层符号(export { I18nProvider, … } from '@object-ui/i18n'),先例在。零破坏。 另加性收 _TOP(导出一个 nonGridRowCeilingQuery(): { $top },调用点再不手写 +1)与重塑 Note(收 result: NonGridCeilingResult,drawnresult.rows.length 推得,旧散 prop 保留为 deprecated 路径)。
                  ⇒ 修好了家与形状,但五个符号仍是公开 API,裁定 a′ 的「internal」并未回复。

                  B —— 回到裁定的字面:收回发布面。 五个符号全部下架,常量落在一个四个插件都能吃到而不经已发布入口的位置。⚠️ 本席读下来这可能做不到:每个包都只有唯一入口,react 无 subpath export ⇒ B 实际要求先给某个包开 subpath export(或新开一个内部包),那本身是更大的一步。若维护者要 B,请一并裁明这一步走哪条。

                  C —— 就地追认。 承认裁定 a′ 的「internal」前提在本仓的包形状下不可达,把五个符号作为公开 API 追认,只做 #7507 的注释订正与 _TOP 的文档降级(标注为派生/测试用)。
                  ⇒ 最省,但把一个「因排期围栏而选中的家」永久冻进公开 API。

                  本席倾向:A

                  理由:(a) 真实需求——core 是四个插件本来就依赖并从中取 extractRecords 的包,家对了代码才读得顺;(b) 长期健全——_TOPNote 的两处加性修整把「不自洽脚注可表达」和「调用点手写 +1」两个真陷阱都关掉,且都零破坏;(c) AI 不易做错——nonGridRowCeilingQuery()+1 只有一处;(d) 创业期尺度——今天零外部消费者,A 是一次性的便宜迁移,B 要先开 subpath export(实现优先、无人拉动),C 则把已知错的家冻住。

                  ⛔ 但 A 不回复裁定 a′ 的字面前提,所以它需要维护者点头而不是本席自裁。

                  派工状态

                  ⛔ 本卡不可派工,needs-user-decision。不带 pm:queue 标签——本卡在裁定前不进队列,这是 #6853 点名的缺陷(卡片自己说不可派工,而标签仍在招手)。

                  不受本卡阻塞、已单独立卡并本轮派工的:#7507(pin 不判定上限、docblock 判别机制写错、changeset 档位与文案、calendar 陈旧 ceiling、假的依赖注释)。审计全文逐字见 #7391 评论 5527977938。

                  Activity

                  Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

                  Metadata

                  Metadata

                  Assignees

                  No one assigned

                    Labels

                    domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpm:queuepriority:p2

                    Type

                    No type

                    Projects

                    No projects

                      Milestone

                      No milestone

                      Relationships

                      None yet

                      Development

                      No branches or pull requests

                      Issue actions

                      , 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
                      Skip to content

                      决策:#7391 在 @object-ui/react 唯一入口上发布了 5 个符号,而裁定 a′ 写的是「the constant is internal」——就地追认、迁 core、还是收回? #7508

                      Description

                      @os-project-manager

                      一句话

                      #7210 的裁定 a′ 逐字写着 「Clause-② no (no contract change; the constant is internal and the footnote is renderer copy)」,而落地的 PR #7391packages/react/src/index.ts发布了五个符号。裁定的前提没有成立,该发布面今天已经在 main 上。⛔ 本席不裁——这是对已发布 API 的处置,且与维护者裁定的字面直接相抵。

                      现在做便宜,以后不便宜:审计实测,除四个渲染器与测试外今天没有任何消费者 import 这五个符号中的任何一个。这个窗口一旦有下游采纳就关上了。

                      发布了什么

                      packages/react/src/index.ts:100-113:

                      export{NON_GRID_ROW_CEILING,NON_GRID_ROW_CEILING_TOP,applyNonGridRowCeiling,NonGridRowCeilingNote,}from'./utils/nonGridRowCeiling.js';exporttype{NonGridCeilingResult}from'./utils/nonGridRowCeiling.js';

                      为什么会走到这里(责任在派发席,记在明处)

                      不是执行车道擅自扩面。那一轮本席自己下的 barrel 围栏把出路堵死了:#7210 评论 5511301369 声明 core#7349types+components#7182 当轮所有,「需要就停下回报」。四个渲染器共用一个常量,唯一没被围住的包是 react,而 react没有 subpath export ⇒ 「把常量放进 react」在物理上只能等于「发布它」。

                      ⚠️ 车道当时应当停下回报(围栏原话如此),而不是铸出 API;但围栏的写法没有点明「若唯一剩下的家是已发布入口,停」。两边各承一半,前一半是本席的。

                      index.ts:100-104 的注释还把这段历史写成了一个实测为假的技术理由——「the only package all four already depend on」;四个插件的 package.json 各自同时依赖 corecomponentstypes。该注释的订正已单独立卡 #7507(仅改注释,⛔ 不动符号)。

                      逐符号的可支撑性(审计读数)

                      符号读数
                      applyNonGridRowCeiling / NonGridCeilingResult可支撑,可加性演进(可选 options 参数、可加字段)。⚠️total 非来源可信:服务端不报 totaldata-objectstackrecords.length 顶替(packages/data-objectstack/src/index.ts:3624-3646),这类服务端上脚注会读成「first 2000 of 2001」。ObjectStack 服务端报 total,故属边界注记。
                      NON_GRID_ROW_CEILING可支撑,但家不对:applyNonGridRowCeilingcore 自己的 extractRecords 的包装,四个插件本来就从 core import 它。
                      NON_GRID_ROW_CEILING_TOP实现细节被发布。它存在只因机制被劈成两半($top 在调用点、切片在 helper)。任何未来的截断探测策略(hasMore、可信 total)都会让它退化成残件。
                      NonGridRowCeilingNote形状是陷阱:收 drawn/total/truncated 三个散 prop,而每个调用点都硬写 drawn={NON_GRID_ROW_CEILING}。组件从不见到它本该渲染的 NonGridCeilingResult ⇒ 不自洽的脚注是可表达的。(组件出现在该入口本身有先例:SchemaRendererI18nProviderElementDataSourceGate。异议在形状,不在类别。)

                      选项

                      A —— 迁 @object-ui/core,react 保留再导出(审计的建议)。NON_GRID_ROW_CEILINGapplyNonGridRowCeilingNonGridCeilingResult 移到 core(extractRecords 旁边),react 继续 export … from '@object-ui/core'——react 早已这样再导出下层符号(export { I18nProvider, … } from '@object-ui/i18n'),先例在。零破坏。 另加性收 _TOP(导出一个 nonGridRowCeilingQuery(): { $top },调用点再不手写 +1)与重塑 Note(收 result: NonGridCeilingResult,drawnresult.rows.length 推得,旧散 prop 保留为 deprecated 路径)。
                      ⇒ 修好了家与形状,但五个符号仍是公开 API,裁定 a′ 的「internal」并未回复。

                      B —— 回到裁定的字面:收回发布面。 五个符号全部下架,常量落在一个四个插件都能吃到而不经已发布入口的位置。⚠️ 本席读下来这可能做不到:每个包都只有唯一入口,react 无 subpath export ⇒ B 实际要求先给某个包开 subpath export(或新开一个内部包),那本身是更大的一步。若维护者要 B,请一并裁明这一步走哪条。

                      C —— 就地追认。 承认裁定 a′ 的「internal」前提在本仓的包形状下不可达,把五个符号作为公开 API 追认,只做 #7507 的注释订正与 _TOP 的文档降级(标注为派生/测试用)。
                      ⇒ 最省,但把一个「因排期围栏而选中的家」永久冻进公开 API。

                      本席倾向:A

                      理由:(a) 真实需求——core 是四个插件本来就依赖并从中取 extractRecords 的包,家对了代码才读得顺;(b) 长期健全——_TOPNote 的两处加性修整把「不自洽脚注可表达」和「调用点手写 +1」两个真陷阱都关掉,且都零破坏;(c) AI 不易做错——nonGridRowCeilingQuery()+1 只有一处;(d) 创业期尺度——今天零外部消费者,A 是一次性的便宜迁移,B 要先开 subpath export(实现优先、无人拉动),C 则把已知错的家冻住。

                      ⛔ 但 A 不回复裁定 a′ 的字面前提,所以它需要维护者点头而不是本席自裁。

                      派工状态

                      ⛔ 本卡不可派工,needs-user-decision。不带 pm:queue 标签——本卡在裁定前不进队列,这是 #6853 点名的缺陷(卡片自己说不可派工,而标签仍在招手)。

                      不受本卡阻塞、已单独立卡并本轮派工的:#7507(pin 不判定上限、docblock 判别机制写错、changeset 档位与文案、calendar 陈旧 ceiling、假的依赖注释)。审计全文逐字见 #7391 评论 5527977938。

                      Activity

                      Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

                      Metadata

                      Metadata

                      Assignees

                      No one assigned

                        Labels

                        domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpm:queuepriority:p2

                        Type

                        No type

                        Projects

                        No projects

                          Milestone

                          No milestone

                          Relationships

                          None yet

                          Development

                          No branches or pull requests

                          Issue actions

                          , 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
                          Skip to content

                          决策:#7391 在 @object-ui/react 唯一入口上发布了 5 个符号,而裁定 a′ 写的是「the constant is internal」——就地追认、迁 core、还是收回? #7508

                          Description

                          @os-project-manager

                          一句话

                          #7210 的裁定 a′ 逐字写着 「Clause-② no (no contract change; the constant is internal and the footnote is renderer copy)」,而落地的 PR #7391packages/react/src/index.ts发布了五个符号。裁定的前提没有成立,该发布面今天已经在 main 上。⛔ 本席不裁——这是对已发布 API 的处置,且与维护者裁定的字面直接相抵。

                          现在做便宜,以后不便宜:审计实测,除四个渲染器与测试外今天没有任何消费者 import 这五个符号中的任何一个。这个窗口一旦有下游采纳就关上了。

                          发布了什么

                          packages/react/src/index.ts:100-113:

                          export{NON_GRID_ROW_CEILING,NON_GRID_ROW_CEILING_TOP,applyNonGridRowCeiling,NonGridRowCeilingNote,}from'./utils/nonGridRowCeiling.js';exporttype{NonGridCeilingResult}from'./utils/nonGridRowCeiling.js';

                          为什么会走到这里(责任在派发席,记在明处)

                          不是执行车道擅自扩面。那一轮本席自己下的 barrel 围栏把出路堵死了:#7210 评论 5511301369 声明 core#7349types+components#7182 当轮所有,「需要就停下回报」。四个渲染器共用一个常量,唯一没被围住的包是 react,而 react没有 subpath export ⇒ 「把常量放进 react」在物理上只能等于「发布它」。

                          ⚠️ 车道当时应当停下回报(围栏原话如此),而不是铸出 API;但围栏的写法没有点明「若唯一剩下的家是已发布入口,停」。两边各承一半,前一半是本席的。

                          index.ts:100-104 的注释还把这段历史写成了一个实测为假的技术理由——「the only package all four already depend on」;四个插件的 package.json 各自同时依赖 corecomponentstypes。该注释的订正已单独立卡 #7507(仅改注释,⛔ 不动符号)。

                          逐符号的可支撑性(审计读数)

                          符号读数
                          applyNonGridRowCeiling / NonGridCeilingResult可支撑,可加性演进(可选 options 参数、可加字段)。⚠️total 非来源可信:服务端不报 totaldata-objectstackrecords.length 顶替(packages/data-objectstack/src/index.ts:3624-3646),这类服务端上脚注会读成「first 2000 of 2001」。ObjectStack 服务端报 total,故属边界注记。
                          NON_GRID_ROW_CEILING可支撑,但家不对:applyNonGridRowCeilingcore 自己的 extractRecords 的包装,四个插件本来就从 core import 它。
                          NON_GRID_ROW_CEILING_TOP实现细节被发布。它存在只因机制被劈成两半($top 在调用点、切片在 helper)。任何未来的截断探测策略(hasMore、可信 total)都会让它退化成残件。
                          NonGridRowCeilingNote形状是陷阱:收 drawn/total/truncated 三个散 prop,而每个调用点都硬写 drawn={NON_GRID_ROW_CEILING}。组件从不见到它本该渲染的 NonGridCeilingResult ⇒ 不自洽的脚注是可表达的。(组件出现在该入口本身有先例:SchemaRendererI18nProviderElementDataSourceGate。异议在形状,不在类别。)

                          选项

                          A —— 迁 @object-ui/core,react 保留再导出(审计的建议)。NON_GRID_ROW_CEILINGapplyNonGridRowCeilingNonGridCeilingResult 移到 core(extractRecords 旁边),react 继续 export … from '@object-ui/core'——react 早已这样再导出下层符号(export { I18nProvider, … } from '@object-ui/i18n'),先例在。零破坏。 另加性收 _TOP(导出一个 nonGridRowCeilingQuery(): { $top },调用点再不手写 +1)与重塑 Note(收 result: NonGridCeilingResult,drawnresult.rows.length 推得,旧散 prop 保留为 deprecated 路径)。
                          ⇒ 修好了家与形状,但五个符号仍是公开 API,裁定 a′ 的「internal」并未回复。

                          B —— 回到裁定的字面:收回发布面。 五个符号全部下架,常量落在一个四个插件都能吃到而不经已发布入口的位置。⚠️ 本席读下来这可能做不到:每个包都只有唯一入口,react 无 subpath export ⇒ B 实际要求先给某个包开 subpath export(或新开一个内部包),那本身是更大的一步。若维护者要 B,请一并裁明这一步走哪条。

                          C —— 就地追认。 承认裁定 a′ 的「internal」前提在本仓的包形状下不可达,把五个符号作为公开 API 追认,只做 #7507 的注释订正与 _TOP 的文档降级(标注为派生/测试用)。
                          ⇒ 最省,但把一个「因排期围栏而选中的家」永久冻进公开 API。

                          本席倾向:A

                          理由:(a) 真实需求——core 是四个插件本来就依赖并从中取 extractRecords 的包,家对了代码才读得顺;(b) 长期健全——_TOPNote 的两处加性修整把「不自洽脚注可表达」和「调用点手写 +1」两个真陷阱都关掉,且都零破坏;(c) AI 不易做错——nonGridRowCeilingQuery()+1 只有一处;(d) 创业期尺度——今天零外部消费者,A 是一次性的便宜迁移,B 要先开 subpath export(实现优先、无人拉动),C 则把已知错的家冻住。

                          ⛔ 但 A 不回复裁定 a′ 的字面前提,所以它需要维护者点头而不是本席自裁。

                          派工状态

                          ⛔ 本卡不可派工,needs-user-decision。不带 pm:queue 标签——本卡在裁定前不进队列,这是 #6853 点名的缺陷(卡片自己说不可派工,而标签仍在招手)。

                          不受本卡阻塞、已单独立卡并本轮派工的:#7507(pin 不判定上限、docblock 判别机制写错、changeset 档位与文案、calendar 陈旧 ceiling、假的依赖注释)。审计全文逐字见 #7391 评论 5527977938。

                          Activity

                          Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

                          Metadata

                          Metadata

                          Assignees

                          No one assigned

                            Labels

                            domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpm:queuepriority:p2

                            Type

                            No type

                            Projects

                            No projects

                              Milestone

                              No milestone

                              Relationships

                              None yet

                              Development

                              No branches or pull requests

                              Issue actions

                              , 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
                              Skip to content

                              决策:#7391 在 @object-ui/react 唯一入口上发布了 5 个符号,而裁定 a′ 写的是「the constant is internal」——就地追认、迁 core、还是收回? #7508

                              Description

                              @os-project-manager

                              一句话

                              #7210 的裁定 a′ 逐字写着 「Clause-② no (no contract change; the constant is internal and the footnote is renderer copy)」,而落地的 PR #7391packages/react/src/index.ts发布了五个符号。裁定的前提没有成立,该发布面今天已经在 main 上。⛔ 本席不裁——这是对已发布 API 的处置,且与维护者裁定的字面直接相抵。

                              现在做便宜,以后不便宜:审计实测,除四个渲染器与测试外今天没有任何消费者 import 这五个符号中的任何一个。这个窗口一旦有下游采纳就关上了。

                              发布了什么

                              packages/react/src/index.ts:100-113:

                              export{NON_GRID_ROW_CEILING,NON_GRID_ROW_CEILING_TOP,applyNonGridRowCeiling,NonGridRowCeilingNote,}from'./utils/nonGridRowCeiling.js';exporttype{NonGridCeilingResult}from'./utils/nonGridRowCeiling.js';

                              为什么会走到这里(责任在派发席,记在明处)

                              不是执行车道擅自扩面。那一轮本席自己下的 barrel 围栏把出路堵死了:#7210 评论 5511301369 声明 core#7349types+components#7182 当轮所有,「需要就停下回报」。四个渲染器共用一个常量,唯一没被围住的包是 react,而 react没有 subpath export ⇒ 「把常量放进 react」在物理上只能等于「发布它」。

                              ⚠️ 车道当时应当停下回报(围栏原话如此),而不是铸出 API;但围栏的写法没有点明「若唯一剩下的家是已发布入口,停」。两边各承一半,前一半是本席的。

                              index.ts:100-104 的注释还把这段历史写成了一个实测为假的技术理由——「the only package all four already depend on」;四个插件的 package.json 各自同时依赖 corecomponentstypes。该注释的订正已单独立卡 #7507(仅改注释,⛔ 不动符号)。

                              逐符号的可支撑性(审计读数)

                              符号读数
                              applyNonGridRowCeiling / NonGridCeilingResult可支撑,可加性演进(可选 options 参数、可加字段)。⚠️total 非来源可信:服务端不报 totaldata-objectstackrecords.length 顶替(packages/data-objectstack/src/index.ts:3624-3646),这类服务端上脚注会读成「first 2000 of 2001」。ObjectStack 服务端报 total,故属边界注记。
                              NON_GRID_ROW_CEILING可支撑,但家不对:applyNonGridRowCeilingcore 自己的 extractRecords 的包装,四个插件本来就从 core import 它。
                              NON_GRID_ROW_CEILING_TOP实现细节被发布。它存在只因机制被劈成两半($top 在调用点、切片在 helper)。任何未来的截断探测策略(hasMore、可信 total)都会让它退化成残件。
                              NonGridRowCeilingNote形状是陷阱:收 drawn/total/truncated 三个散 prop,而每个调用点都硬写 drawn={NON_GRID_ROW_CEILING}。组件从不见到它本该渲染的 NonGridCeilingResult ⇒ 不自洽的脚注是可表达的。(组件出现在该入口本身有先例:SchemaRendererI18nProviderElementDataSourceGate。异议在形状,不在类别。)

                              选项

                              A —— 迁 @object-ui/core,react 保留再导出(审计的建议)。NON_GRID_ROW_CEILINGapplyNonGridRowCeilingNonGridCeilingResult 移到 core(extractRecords 旁边),react 继续 export … from '@object-ui/core'——react 早已这样再导出下层符号(export { I18nProvider, … } from '@object-ui/i18n'),先例在。零破坏。 另加性收 _TOP(导出一个 nonGridRowCeilingQuery(): { $top },调用点再不手写 +1)与重塑 Note(收 result: NonGridCeilingResult,drawnresult.rows.length 推得,旧散 prop 保留为 deprecated 路径)。
                              ⇒ 修好了家与形状,但五个符号仍是公开 API,裁定 a′ 的「internal」并未回复。

                              B —— 回到裁定的字面:收回发布面。 五个符号全部下架,常量落在一个四个插件都能吃到而不经已发布入口的位置。⚠️ 本席读下来这可能做不到:每个包都只有唯一入口,react 无 subpath export ⇒ B 实际要求先给某个包开 subpath export(或新开一个内部包),那本身是更大的一步。若维护者要 B,请一并裁明这一步走哪条。

                              C —— 就地追认。 承认裁定 a′ 的「internal」前提在本仓的包形状下不可达,把五个符号作为公开 API 追认,只做 #7507 的注释订正与 _TOP 的文档降级(标注为派生/测试用)。
                              ⇒ 最省,但把一个「因排期围栏而选中的家」永久冻进公开 API。

                              本席倾向:A

                              理由:(a) 真实需求——core 是四个插件本来就依赖并从中取 extractRecords 的包,家对了代码才读得顺;(b) 长期健全——_TOPNote 的两处加性修整把「不自洽脚注可表达」和「调用点手写 +1」两个真陷阱都关掉,且都零破坏;(c) AI 不易做错——nonGridRowCeilingQuery()+1 只有一处;(d) 创业期尺度——今天零外部消费者,A 是一次性的便宜迁移,B 要先开 subpath export(实现优先、无人拉动),C 则把已知错的家冻住。

                              ⛔ 但 A 不回复裁定 a′ 的字面前提,所以它需要维护者点头而不是本席自裁。

                              派工状态

                              ⛔ 本卡不可派工,needs-user-decision。不带 pm:queue 标签——本卡在裁定前不进队列,这是 #6853 点名的缺陷(卡片自己说不可派工,而标签仍在招手)。

                              不受本卡阻塞、已单独立卡并本轮派工的:#7507(pin 不判定上限、docblock 判别机制写错、changeset 档位与文案、calendar 陈旧 ceiling、假的依赖注释)。审计全文逐字见 #7391 评论 5527977938。

                              Activity

                              Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

                              Metadata

                              Metadata

                              Assignees

                              No one assigned

                                Labels

                                domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpm:queuepriority:p2

                                Type

                                No type

                                Projects

                                No projects

                                  Milestone

                                  No milestone

                                  Relationships

                                  None yet

                                  Development

                                  No branches or pull requests

                                  Issue actions