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
[finding] Two more source comments name out-of-repo app paths as live — objectql/src/plugin.ts:142 and cloud-connection/.../marketplace-install-local-seed-replayer.test.ts:16, both missed by #14806's census #15297
Found while implementing #14806 (dev seat, session session_01D47qPfEWVPmhguWgBZCi5N, branch claude/issue-14806-out-of-repo-app-path-comments). Filed unassigned for triage. Deliberately NOT swept by that branch — both sites are outside the file surface that branch declared before editing, and one is in another package family, so editing them would have made a declared claim untrue while another lane was reading it.
The class, already established
A source comment naming a repository path is a checkable claim. apps/objectos and apps/cloud were deleted from this repo and now live in objectstack-ai/cloud. #14634 fixed three such sites, #14806 fixed six more. These two are the same defect and were in neither census.
Positive control, because the claim is a negative — apps/ is not empty and the query is not broken:
$ git ls-tree origin/main -- apps/
040000 tree f281e0156f1e4569560b79b9899d3ea56cfb7f1d apps/docs
Site 1 — packages/objectql/src/plugin.ts:142
TSDoc on the skipSchemaSync option. Present tense, unmarked:
/***Skipboth`syncRegisteredSchemas()`callsinside`start()`and*assumeDDL is managedout-of-band(e.g.an`apps/cloud/scripts/migrate.ts`*runbeforedeploythatconnectsdirectlytothedatabaseandcreates*all`sys_*`+customtablesonce).
⭐ This is the same sentence, about the same script, as packages/cli/src/commands/serve.ts's migrate-and-exit note — which #14806's branch has just fixed. They were written as a pair and drifted apart because only one of them was ever in a census. The referent is genuine, so the disposition is to mark it out-of-repo, not to rename or delete it.
Site 2 — packages/cloud-connection/src/marketplace-install-local-seed-replayer.test.ts:16
// A runtime that declares no data of its own — `objects: []`, no `data`, which is// exactly what a marketplace-install target looks like and exactly what// `apps/objectos-ee` is — therefore ended up with `seed-datasets` POPULATED and// `seed-replayer` ABSENT.
Weaker than site 1 — it is a test's rationale, not an instruction — but it is the only apps/objectos-ee mention in the tree that does not mark the app as living elsewhere. Its three siblings all do: service-cluster/src/multi-node-gate-mount.ts:9 ("cloud repo"), service-cluster/src/multi-node-gate.test.ts:227 ("cloud: …"), packages/types/src/node.ts:72 ("Measured on cloud's …"). Consistency is the whole value here.
The spelling to copy — do not invent a fourth one
Take it from the branch above, so the sweep ends with one house style rather than four. serve.ts's migrate-and-exit note now reads:
// Out-of-band migration mode: the caller (e.g.// `apps/cloud/scripts/migrate.ts`, which lives in the separate// `objectstack-ai/cloud` repo and is NOT a path in this one) just// wants the kernel bootstrap (ObjectQLPlugin → schema sync →
The oldest in-repo precedent is multi-node-gate-mount.ts:9's terse "(apps/objectos-ee/objectstack.config.ts, cloud repo)". Either is fine; a third is not.
The trap, unchanged and still the whole risk
⛔ Grep for the pathsapps/objectos and apps/cloud. ⛔ Never for the bare tokens objectos or cloud — objectos.ai, cloud.objectos.ai, DEFAULT_CLOUD_URL, the root-domain guard, admin@objectos.ai as the dev seed address and the plugin ids objectos-ee / objectos-runtime / objectos-environment / objectos-stack are all correct, several matched as literal plugin ids inside serve.ts. A token sweep breaks working code.
⛔ Do not add an apps/objectos or apps/cloud directory to this repo to satisfy any comment.
packages/observability/README.md:39 is listed for completeness and is not asserted as a defect: it already reads "handled in apps/cloud — see that repo's exporter", which does mark the referent as out-of-repo, just without naming the repo. A seat may tighten it while here; nothing is wrong if it does not.
packages/spec/prompts/architecture.md:26 is a product-strategy inventory listing apps/www / apps/marketplace / apps/studio beside apps/cloud — none of those is in this repo either, so it is not making a claim about this tree.
The _console/apps/cloud_control/... matches in cloud-connection/README.md, cloud-connection/src/runtime-config-upgrade-url.test.ts and spec/src/ui/inline-action.test.ts are a different path (a Console route), and every CHANGELOG.md, ROADMAP.md and docs/adr/** hit is a historical record. ⛔ Never rewrite those.
Found while implementing #14806 (dev seat, session
session_01D47qPfEWVPmhguWgBZCi5N, branchclaude/issue-14806-out-of-repo-app-path-comments). Filed unassigned for triage. Deliberately NOT swept by that branch — both sites are outside the file surface that branch declared before editing, and one is in another package family, so editing them would have made a declared claim untrue while another lane was reading it.The class, already established
A source comment naming a repository path is a checkable claim.
apps/objectosandapps/cloudwere deleted from this repo and now live inobjectstack-ai/cloud. #14634 fixed three such sites, #14806 fixed six more. These two are the same defect and were in neither census.Positive control, because the claim is a negative —
apps/is not empty and the query is not broken:Site 1 —
packages/objectql/src/plugin.ts:142TSDoc on the
skipSchemaSyncoption. Present tense, unmarked:⭐ This is the same sentence, about the same script, as
packages/cli/src/commands/serve.ts's migrate-and-exit note — which #14806's branch has just fixed. They were written as a pair and drifted apart because only one of them was ever in a census. The referent is genuine, so the disposition is to mark it out-of-repo, not to rename or delete it.Site 2 —
packages/cloud-connection/src/marketplace-install-local-seed-replayer.test.ts:16Weaker than site 1 — it is a test's rationale, not an instruction — but it is the only
apps/objectos-eemention in the tree that does not mark the app as living elsewhere. Its three siblings all do:service-cluster/src/multi-node-gate-mount.ts:9("cloud repo"),service-cluster/src/multi-node-gate.test.ts:227("cloud: …"),packages/types/src/node.ts:72("Measured on cloud's …"). Consistency is the whole value here.The spelling to copy — do not invent a fourth one
Take it from the branch above, so the sweep ends with one house style rather than four.
serve.ts's migrate-and-exit note now reads:The oldest in-repo precedent is
multi-node-gate-mount.ts:9's terse "(apps/objectos-ee/objectstack.config.ts, cloud repo)". Either is fine; a third is not.The trap, unchanged and still the whole risk
⛔ Grep for the paths
apps/objectosandapps/cloud. ⛔ Never for the bare tokensobjectosorcloud—objectos.ai,cloud.objectos.ai,DEFAULT_CLOUD_URL, the root-domain guard,admin@objectos.aias the dev seed address and the plugin idsobjectos-ee/objectos-runtime/objectos-environment/objectos-stackare all correct, several matched as literal plugin ids insideserve.ts. A token sweep breaks working code.⛔ Do not add an
apps/objectosorapps/clouddirectory to this repo to satisfy any comment.Explicitly NOT in this card
packages/cli/src/commands/package/publish.ts:168stays on [finding] Seven more source comments name deleted in-repo app paths (apps/objectos,apps/cloud) as live — same class as #14634, outside its ruled scope #14806 — it is a CLI help example string, not a comment, and that card's dispatch was ruled to comment text only. It is the remainder there, which is why that branch's PR saysPart ofrather than a closing keyword.packages/cli/src/commands/serve.ts:133is [question]serve.ts:133still orders "keep the two in sync" withapps/cloud/server/observability.ts— does that mirror still exist inobjectstack-ai/cloud? #15295 — a live cross-repo sync obligation, a decision rather than a rename.packages/observability/README.md:39is listed for completeness and is not asserted as a defect: it already reads "handled in apps/cloud — see that repo's exporter", which does mark the referent as out-of-repo, just without naming the repo. A seat may tighten it while here; nothing is wrong if it does not.packages/spec/prompts/architecture.md:26is a product-strategy inventory listingapps/www/apps/marketplace/apps/studiobesideapps/cloud— none of those is in this repo either, so it is not making a claim about this tree._console/apps/cloud_control/...matches incloud-connection/README.md,cloud-connection/src/runtime-config-upgrade-url.test.tsandspec/src/ui/inline-action.test.tsare a different path (a Console route), and everyCHANGELOG.md,ROADMAP.mdanddocs/adr/**hit is a historical record. ⛔ Never rewrite those.Refs: #14806, #14634, #15295.