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
objectui#6809 (PR pending) narrows packages/components' browser process shim so it no longer reaches the package's TEST project: the declaration moved from src/global.d.ts to src/browser-process-shim.d.ts, and packages/components/tsconfig.test.json names that one file in exclude. With that in place, tsc -p tsconfig.test.json resolves process to node's real global again — measured green on all of process.cwd(), process.platform and process.versions.node.
PR #6808 (claude/issue-6779-container-declaration-ratchet, still open and unmerged) predates that and carries a local workaround at its call site, with the reasoning written out beside it:
Once #6808 lands, that cast is dead weight — the plain process.cwd() idiom compiles in this package, and the workaround now reads as a warning about a hazard that no longer exists.
Why it was not done in the objectui#6809 PR
The call site does not exist on main yet: #6808 is unmerged, so there is nothing in the objectui#6809 branch's base to edit. Touching it there would have meant either editing a file that only exists on another agent's branch or creating a merge conflict with in-flight work. The objectui#6809 dispatch ruled it out explicitly and asked for this follow-up instead.
Scope
Replace the globalThis cast at its call site with the plain process.cwd() spelling.
Remove the explanatory comment that justifies the cast, or reduce it to a pointer at objectui#6809 if the surrounding prose still needs one.
Blocked-by: #6808
What
objectui#6809 (PR pending) narrows
packages/components' browserprocessshim so it no longer reaches the package's TEST project: the declaration moved fromsrc/global.d.tstosrc/browser-process-shim.d.ts, andpackages/components/tsconfig.test.jsonnames that one file inexclude. With that in place,tsc -p tsconfig.test.jsonresolvesprocessto node's real global again — measured green on all ofprocess.cwd(),process.platformandprocess.versions.node.PR #6808 (
claude/issue-6779-container-declaration-ratchet, still open and unmerged) predates that and carries a local workaround at its call site, with the reasoning written out beside it:Once #6808 lands, that cast is dead weight — the plain
process.cwd()idiom compiles in this package, and the workaround now reads as a warning about a hazard that no longer exists.Why it was not done in the objectui#6809 PR
The call site does not exist on
mainyet: #6808 is unmerged, so there is nothing in the objectui#6809 branch's base to edit. Touching it there would have meant either editing a file that only exists on another agent's branch or creating a merge conflict with in-flight work. The objectui#6809 dispatch ruled it out explicitly and asked for this follow-up instead.Scope
globalThiscast at its call site with the plainprocess.cwd()spelling.Generated by Claude Code