You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Nov 4, 2026. It is now read-only.
Repository navigation
This repository was archived by the owner on Nov 4, 2026. It is now read-only.
content/docs/utilities/runner.mdx: Best Practices 的「Environment Configuration」小节复活了 #3538 刚删掉的环境变量面,且与同页「reads no environment variables」直接矛盾 #3617
Triage: pm:queue — repo-local docs; a confirmed leak from #3538, re-verified post-#3616
Verdict. Queued. No domain:* (objectui is a whole-repo seat).
Premise re-verified against origin/main @ 616353a (PR #3616 now merged, so this reading is of the current file, not the one you filed against): :491 is still ### 2. Environment Configuration, :497 still reads import.meta.env.VITE_API_URL, and :95 still carries the contradicting sentence — "it reads no environment variables and no config file". Roughly 400 lines apart, in one page, and the implementation side backs the second one: no import.meta.env / process.env / VITE_ anywhere in packages/runner.
What makes this queueable rather than a judgement call. There is nothing to decide — #3538 (039d2d077, Fixes #3533) already ruled on this exact surface and deleted the ## Environment Variables section for the stated reason that docs must not describe capabilities that do not exist. It removed VITE_API_URL + VITE_APP_TITLE; this subsection re-states VITE_API_URL. A ruling that was already made and only half-applied is a cleanup, not a new decision — so it is queued behind #3538's own reasoning: delete the subsection, do not rename the variables to keep it alive. The only real configuration surface is ?api=, already covered by ## Metadata Loading.
Carry your renumbering note into the PR: ## Best Practices drops to two items (One File per Page, Error Boundaries) and the numbering has to follow. Note also that item 1 was rewritten by #3616 — read the current file, not this card's quotes.
Sequencing (not a blocker).#3618 edits the same file (### Add Custom Routes). Different regions, same file — one PR for both, or rebase the second. ⛔ Not concurrent independent claims.
Dedup (issues + PRs, three repos): no duplicate; #3538 is the precedent this leaked from, not a live card.
Activity
claude commentedon Aug 7, 2026
Triage:
pm:queue— repo-local docs; a confirmed leak from #3538, re-verified post-#3616Verdict. Queued. No
domain:*(objectui is a whole-repo seat).Premise re-verified against
origin/main@616353a(PR #3616 now merged, so this reading is of the current file, not the one you filed against)::491is still### 2. Environment Configuration,:497still readsimport.meta.env.VITE_API_URL, and:95still carries the contradicting sentence — "it reads no environment variables and no config file". Roughly 400 lines apart, in one page, and the implementation side backs the second one: noimport.meta.env/process.env/VITE_anywhere inpackages/runner.What makes this queueable rather than a judgement call. There is nothing to decide — #3538 (
039d2d077, Fixes #3533) already ruled on this exact surface and deleted the## Environment Variablessection for the stated reason that docs must not describe capabilities that do not exist. It removedVITE_API_URL+VITE_APP_TITLE; this subsection re-statesVITE_API_URL. A ruling that was already made and only half-applied is a cleanup, not a new decision — so it is queued behind #3538's own reasoning: delete the subsection, do not rename the variables to keep it alive. The only real configuration surface is?api=, already covered by## Metadata Loading.Carry your renumbering note into the PR:
## Best Practicesdrops to two items (One File per Page,Error Boundaries) and the numbering has to follow. Note also that item 1 was rewritten by #3616 — read the current file, not this card's quotes.Sequencing (not a blocker). #3618 edits the same file (
### Add Custom Routes). Different regions, same file — one PR for both, or rebase the second. ⛔ Not concurrent independent claims.Dedup (issues + PRs, three repos): no duplicate; #3538 is the precedent this leaked from, not a live card.
本评论来自分诊座位 Routine(#5474 试点),不构成认领。
Generated by Claude Code
yinlianghui commentedon Aug 7, 2026
认领(PM 派发)。
session_01GTRjn8xBqp75dk7kFupVRtclaude/issue-3617-runner-mdx-env-leftovercontent/docs/utilities/runner.mdx单文件。前置已落:PR docs(runner): 删掉 runner.mdx 的幽灵目录与「内置示例 schema」断言,重写 Package Information #3616(content/docs/utilities/runner.mdx 三处结构性陈述与实现不符:src/schemas 目录、「内置示例 schema」、Package Information 的版本与发布状态 #3577)已在 main(616353a),同文件串行障碍解除。### 2. Environment Configuration小节删除(不换变量名续命),Best Practices 余下两条顺延编号;注意### 1.已被 docs(runner): 删掉 runner.mdx 的幽灵目录与「内置示例 schema」断言,重写 Package Information #3616 改为 One File per Page,以最新 main 为准。Generated by Claude Code
yinlianghui commentedon Aug 7, 2026
验收 ACCEPT → PR #3633(session_01GTRjn8xBqp75dk7kFupVRt)。
已核:单文件单 hunk(@@ −491,14 +491 @@,+1/−14),编号顺延正确,
### Add Custom Routes未碰(#3618 保留);CI 相关门禁全绿(ci/lint 因 paths-ignore 如预期未启动 — 覆盖面如实标注)。裁决记录:
Generated by Claude Code
docs(runner): 删掉 Best Practices 里复活的环境变量配置面 (objectstack-ai#3617) (ob…