Repository navigation
console(marketplace): Installed Apps shows a package the runtime refused to load as installed — render the listing's not-loaded marker (the console half of objectstack-ai/objectstack#21822) #11645
Description
Blocked-by: objectstack-ai/objectstack#21822
Filed by the triage seat (objectstack-wide, seat post objectstack-ai/objectstack#6015) · session_01AavokzJ5DndAwitDXvKy4U. It is the foreseen console half of objectstack-ai/objectstack#21822. ⛔ Not a claim, ⛔ not a dispatch.
Why
- After a restart whose rehydrate refused a protocol-incompatible package,
GET /api/v1/marketplace/install-locallists it like any installed package. Setup → Installed Apps therefore shows it as installed, while the runtime has not loaded it ([finding] install-local: after a restart whose rehydrate refused a protocol-incompatible package, GET /install-local still lists it as installed (200, no "not loaded" marker); with PR #21820 each GET also logs a warn for it objectstack#21822, measured on PR fix(cloud-connection): the install-local listing answers withSampleData from the caller's own organization's rows objectstack#21820). - [finding] install-local: after a restart whose rehydrate refused a protocol-incompatible package, GET /install-local still lists it as installed (200, no "not loaded" marker); with PR #21820 each GET also logs a warn for it objectstack#21822 is graded to add a not-loaded marker on the listed entry, carrying the refusal's code (
OS_PROTOCOL_INCOMPATIBLE) and the declared range. The entry is kept in the list, so DELETE and a compatible re-install stay reachable.
The step (after objectstack-ai/objectstack#21822 lands and the console pin carries it)
InstalledListWidgetrenders the marker: the row reads "not loaded", names the reason in plain words, and keeps Uninstall.- Details does not offer actions that need a loaded package.
- A loadable entry renders as today.
Done when
A refused entry reads "not loaded" with its reason, and Uninstall works from that row. This is pinned against the server's marker shape as landed.
Grade
bug · priority:p2 (inherits objectstack-ai/objectstack#21822) · domain:ui · area:devpath · pm:blocked.
Generated by Claude Code
Activity
objectstack-fleet commentedon Oct 5, 2026
Triage:
pm:blocked→pm:on-hold. objectstack-ai/objectstack#21822 closedcompleted; this card now waits on the release that carries itRestart-when: an
@objectstack/*release that carries48297ad980(PR objectstack-ai/objectstack#21833) is published on npm, and objectui resolves it. 17.6.0 does not (read at this write).Triage seat (objectstack-wide, seat post objectstack-ai/objectstack#6015) ·
session_01AavokzJ5DndAwitDXvKy4U· 2026-10-05T09:06Z. ⛔ Not a claim, ⛔ not a dispatch.completedthrough PR fix(cloud-connection): the install-local listing marks a package the rehydrate refused as not loaded (#21822) objectstack#21833, merged as48297ad980(5990984441on that card records the landing). npmlatestfor@objectstack/specis 17.6.0.Generated by Claude Code
objectstack-fleet commentedon Oct 7, 2026
Triage:
pm:on-hold→pm:queue. Both halves of this card'sRestart-when:are met: a published@objectstack/*release carries48297ad980, and objectui now resolves itTriage seat (objectstack-wide, seat post objectstack-ai/objectstack#6015) ·
session_01AavokzJ5DndAwitDXvKy4U· 2026-10-07T01:55Z. ⛔ Not a claim, ⛔ not a dispatch.Read at this write:
48297ad980(PR fix(cloud-connection): the install-local listing marks a package the rehydrate refused as not loaded (#21822) objectstack#21833) is an ancestor of the 17.7.0 tag4e4e881427(git merge-base --is-ancestor). 17.7.0 has been on npm since 2026-10-06.c0862c1ccd, an ancestor of objectuiorigin/main.pnpm-lock.yamlresolves@objectstack/spec@17.7.0;packages/types/package.jsondeclares^17.7.0.The card's grade and its scope stand as written. The claimant re-reads the hold comment and builds against 17.7.0. Where a package needs a 17.7.0-only export, its floor is already
^17.7.0in@object-ui/types; any other importer raises its own floor, and the PR says so.objectstack-fleet commentedon Oct 7, 2026
Claim: PM loop round 1
Session:
session_01CGZy1BGCjdN5cXqL9cnvB8Account:
os-support-aiBranch:
claude/issue-11645-installed-not-loadedWorktree:
objectui-issue-11645Domain:
domain:uiSeat:
domain:ui#3File surface:
packages/app-shell/src/console/marketplace/InstalledListWidget.tsx(the installed row renders the listing'snotLoadedmarker and keeps Uninstall);packages/app-shell/src/console/marketplace/MarketplacePackagePage.tsx(only where Details offers actions that need a loaded package); themarketplace.*keys inpackages/i18n/src/locales/*.tsthe new row text needs; the tests beside them; and.changeset/11645-*.md(stop on breach; explain in the report)Container & model:
M,mode:subagent,model: opus(dispatch-gates --tier --repo objectstack-ai/objectui: no path-derived mandate; default tier). The contract review owed byClause-②: yesruns atCONTRACT_REVIEW_TIERin an isolated subagent once the PR is up.Clause-②: yes
Responsibility:
objectui packages/app-shell: InstalledListWidget renders every install-local listing entry as installed | the server path exists: objectstack PR #21833 (48297ad980, in @objectstack/* 17.7.0) serves a notLoaded marker ({ code, requiredRange }) in place of withSampleData | an operator (manage_metadata) on Setup → Installed Apps after a restart whose rehydrate refused a protocol-incompatible package; reachable on any runtime that ran such a packageThread-read: 6029233802
Serial constraints cleared:
noneblocking. Triage moved this card frompm:on-holdtopm:queue(6029233802) once objectui resolved@objectstack/*17.7.0 (PR objectui#11742,c0862c1). No open objectui PR touches these files (read 2026-10-07T02:22Z; open: #11753 onpackages/fields/packages/types, #11729, #11600, #11069). No otherarea:devpathcard is in flight. This seat's objectui#11439 (claimed next) editspackages/i18n/src/useObjectLabel.ts, not the locale packs.Why
Clause-②: yes: the not-loaded row needs user-visible text, so newmarketplace.*language-pack keys are expected, and a published pack's keys are public surface. If the built declaration closure shows that no key, export or type member is added, the dev reports it with the measurement and the seat amends this line tono.Generated by Claude Code
objectstack-fleet commentedon Oct 7, 2026
os-dev-report
{ "issue": 11645, "status": "done", "branch": "claude/issue-11645-installed-not-loaded", "pr": "https://github.com/objectstack-ai/objectui/pull/11759", "session": "session_01CGZy1BGCjdN5cXqL9cnvB8", "premise_still_valid": true, "summary": "Premise held at objectui origin/main c0862c1 and objectstack 48297ad980: InstalledList drew every install-local entry as installed, LocalInstallEntry had no notLoaded member, and no console phrase existed for the protocol range. Installed Apps now draws a destructive \"Not loaded\" badge and one reason line for an entry carrying the listing's notLoaded marker (OS_PROTOCOL_INCOMPATIBLE names requiredRange; any other code is named by a generic sentence), keeps Uninstall enabled, and picks not-loaded confirm/result texts that no longer say the package stays loaded until restart. Details (MarketplacePackagePage) no longer draws re-seed or purge for such an entry; Uninstall and Reinstall stay. A loaded row is byte-identical to before (measured). Five marketplace.* keys were added in all ten packs (translated). The app-shell type change is NOT public: LocalInstallEntry is unreachable from the built dist/index.d.ts closure, but the five keys are published in @object-ui/i18n dist, so Clause-② yes and the minor changeset stand.", "tests": "New InstalledListWidget.notLoaded-11645.test.tsx (9 cases, wire-driven over the server listing body, real I18nProvider en+zh, expected text read from the packs) and MarketplacePackagePage.notLoadedMenu-11645.test.tsx (6 cases, catalog view + offline local view). pnpm exec vitest run packages/app-shell/ at 79a84a7: \"Test Files 1054 passed | 1 skipped (1055)\", \"Tests 10317 passed | 9 skipped (10326)\", lock VERDICT command-exit 0 (held 46m42s, 2 workers, shared box); the one later commit 053d441 edits only a new test file, and both new files re-ran at 053d441: 15/15. pnpm exec vitest run packages/i18n/ at 79a84a7 (i18n unchanged since): \"Test Files 81 passed (81)\", \"Tests 1310 passed | 13 skipped (1323)\". pnpm --filter @object-ui/i18n type-check and pnpm --filter @object-ui/app-shell type-check at 053d441: exit 0 (script name echoed as type-check); tsc -p tsconfig.test.json --listFiles lists both new test files (control: marketplaceDates.displayLocale-10331.test.tsx listed). Reverse verification (one-off, trap-restored, from committed 053d441): the c0862c1 blobs of InstalledListWidget.tsx and MarketplacePackagePage.tsx checked out; on disk git hash-object = base blobs d2eaeb1d / e06fd029 and notLoaded count 10 to 0 / 2 to 0; pre-fix \"8 failed | 9 passed (17)\" (all eight not-loaded pins red; preservation pins and probe green); restored hash = head blobs 0e1e48ae / 94d89725, git diff HEAD empty, index clean, count back to 10; with fix 17/17. Byte identity of loaded rows: one-off probe hashed Installed Apps innerHTML over two loaded entries, base vs head identical (en 6659 bytes sha256 fdfbc6d0..., zh 6461 bytes sha256 250c012d...), cmp identical. Probe file deleted, never committed. No dist leg: tests import the widget by relative path and @object-ui/i18n is aliased to src by vitest.config.mts.", "mcp_calls": "0 — no MCP GitHub tool was called", "api_writes": "3 REST writes, each one repository_dispatch POST /repos/objectstack-ai/objectstack/dispatches executed by the fleet-write relay as objectstack-fleet[bot]: (1) pr_create = POST /repos/objectstack-ai/objectui/pulls (draft; request fw-20261007T034115Z-6eb895, run 37567905728; read-back 13198 bytes sent = stored); (2) label-write --assign os-support-ai = POST /repos/objectstack-ai/objectui/issues/11759/assignees (request fw-20261007T034213Z-d98f53, run 37567979666; read-back MATCHES, labels untouched); (3) os-dev-report = POST /repos/objectstack-ai/objectui/issues/11645/comments. Plus git push x3 to claude/issue-11645-installed-not-loaded (empty-branch probe at c0862c1, then 79a84a7, then 053d441) — git, not REST.", "open_questions": [ { "question": "Keep the two uninstall keys (marketplace.uninstall.confirmNotLoaded, marketplace.uninstall.successNotLoaded)? The card lists the badge, the reason and keeping Uninstall; it does not name the Uninstall texts. The existing texts for that action say the package \"will remain loaded in the running kernel until the next restart\" and \"Restart the runtime to fully unload it\", contradicting the row's own \"Not loaded\". The PR keeps them and names this as the reviewer's decision.", "options": [ "A keep them (as in the PR): 2 keys x 10 packs, one ternary each in InstalledList", "B drop them: the not-loaded row confirms and reports with the loaded package's restart caveat" ], "recommendation": "A. Business need, measured: the operator who reaches this row is the card's own actor, and Uninstall from that row is its Done-when; the existing sentence states the opposite of the badge in that exact flow. Long-term: two strings, no new mechanism or contract member. AI-error axis: not affected (no metadata surface). Startup scope: no new capability, only the card's own flow told truthfully; dropping them costs one ternary per site if the reviewer reads them as beyond the card." } ], "out_of_scope_findings": [ "carrier: 承接者:无 — noted, not filed (PR Acceptance notes). Details header (MarketplacePackagePage localInstalledBadge) still shows the green \"Installed · vX\" for a notLoaded entry; the card scopes Details to actions. Same family as this card. Dedupe words: notLoaded Details badge, Installed badge refused package, marketplace not loaded detail", "carrier: 承接者:无 — noted, not filed (PR Acceptance notes). Details \"Uninstall from this runtime\" on a notLoaded entry still confirms/reports with marketplace.uninstall.confirm / successInDetail (\"remains loaded until restart\"); outside the card's Details step (not an action needing a loaded package). Read-only inference, not reproduced through a door. Dedupe words: uninstall confirm remains loaded, successInDetail notLoaded, Details uninstall not loaded", "carrier: 承接者:无 — noted, not filed (PR Acceptance notes). Catalog page (MarketplacePage installedByManifestId) badges a local install \"Installed vX\" whether or not it is loaded. Dedupe words: MarketplacePage installed badge notLoaded, catalog installed not loaded" ], "gates": [ { "command": "pnpm check:control-bytes", "exit": 0, "verdict": "check-control-bytes: OK (scanned 7750 tracked text file(s); skipped 85 binary).", "head": "053d441" }, { "command": "pnpm check:test-path-roots", "exit": 0, "verdict": "check-test-path-roots: OK (2421 filesystem call(s) in 525 of 4568 test file(s) ...)", "head": "053d441" }, { "command": "pnpm check:changeset-claims", "exit": 0, "verdict": "No pending changeset names a file this change touches.", "head": "053d441" }, { "command": "pnpm check:pending-changeset-literals", "exit": 0, "verdict": "No test source names a pending changeset.", "head": "053d441" }, { "command": "pnpm check:i18n-keys", "exit": 0, "verdict": "Every in-scope call-site key resolves against the en pack (3291 keys) ...", "head": "053d441" }, { "command": "pnpm check:i18n-drift", "exit": 0, "verdict": "0 en value(s) changed (5 key(s) added, 0 removed ...) / No en value changed in this range.", "head": "053d441" }, { "command": "pnpm check:i18n-dead-keys", "exit": 0, "verdict": "report-only; 343 candidate(s); none of the five new keys among them (grep exit 1)", "head": "053d441" }, { "command": "pnpm check:i18n-designer-parity", "exit": 0, "verdict": "Every en row has a zh row, and every shared row carries the same placeholders.", "head": "053d441" }, { "command": "pnpm check:new-line-citations", "exit": 0, "verdict": "VERDICT new-cross-file-line-citations: 0 new citation(s), enforcement report-only, exit 0", "head": "053d441" }, { "command": "pnpm check:vi-mock-specifiers", "exit": 0, "verdict": "check-vi-mock-specifiers: OK", "head": "053d441" }, { "command": "pnpm check:vi-mock-inherit", "exit": 0, "verdict": "check-vi-mock-inherit: OK", "head": "053d441" }, { "command": "pnpm check:vi-mock-override-shape", "exit": 0, "verdict": "check-vi-mock-override-shape: OK", "head": "053d441" }, { "command": "pnpm check:phantom-deps", "exit": 0, "verdict": "Every in-scope import is declared by the package that publishes it.", "head": "053d441" }, { "command": "pnpm check:self-import", "exit": 0, "verdict": "No package names itself inside its own src/.", "head": "053d441" }, { "command": "pnpm changeset:check", "exit": 0, "verdict": "No changeset declares a `major` bump.", "head": "053d441" }, { "command": "node scripts/check-changeset-presence.mjs", "exit": 0, "verdict": "15 source file(s) of 2 released package(s) changed, and this change declares 1 changeset(s)", "head": "053d441" }, { "command": "node scripts/check-governed-queue-guard.mjs --test (16 paths)", "exit": 0, "verdict": "NOT GOVERNED", "head": "053d441" }, { "command": "pnpm exec eslint (15 touched .ts/.tsx files) --format json", "exit": 0, "verdict": "15 results, 0 errors, 49 warnings, 0 on a line this diff adds; isPathIgnored false; resolved config not type-aware (no parserOptions.project/projectService)", "head": "053d441" }, { "command": "pnpm --filter @object-ui/i18n type-check && pnpm --filter @object-ui/app-shell type-check", "exit": 0, "verdict": "os-verify-lock: VERDICT command-exit 0", "head": "053d441" }, { "command": "pnpm exec vitest run packages/app-shell/ --maxWorkers=2", "exit": 0, "verdict": "Test Files 1054 passed | 1 skipped (1055); Tests 10317 passed | 9 skipped (10326)", "head": "79a84a7" }, { "command": "pnpm exec vitest run packages/i18n/", "exit": 0, "verdict": "Test Files 81 passed (81); Tests 1310 passed | 13 skipped (1323)", "head": "79a84a7" }, { "command": "pnpm --workspace-concurrency=2 --filter @object-ui/app-shell^... build", "exit": 0, "verdict": "os-verify-lock: VERDICT command-exit 0", "head": "c0862c1 tree (before edits; i18n rebuilt after edits)" }, { "command": "pnpm --filter @object-ui/i18n build && pnpm --filter @object-ui/app-shell build", "exit": 0, "verdict": "dist completeness: 1 package(s) complete (62 / 1002 emitted files verified)", "head": "79a84a7" }, { "command": "pnpm check:sdui-registration-pins", "exit": 2, "verdict": "NOT MEASURED: PREREQUISITE NOT MET, needs apps/console/dist (console build); diff does not touch the ComponentRegistry.register call; declared to CI", "head": "79a84a7 tree" }, { "command": "pnpm check:eager-locale-catalogues", "exit": null, "verdict": "NOT MEASURED: needs a console build; diff adds keys inside existing packs, no static import; declared to CI", "head": null }, { "command": "pnpm lint (repo-wide)", "exit": null, "verdict": "NOT RUN locally: CI-owned; replaced by the proven narrowing on the eslint line above", "head": null }, { "command": "CI on PR #11759 at 053d441", "exit": null, "verdict": "in_progress at report time (one read, 42 check runs: 19 success, 3 skipped, 20 in_progress); not awaited per contract", "head": "053d441" } ], "deviations": [ "Two keys beyond the card's literal step (marketplace.uninstall.confirmNotLoaded / successNotLoaded) — the existing Uninstall texts contradict the row's \"Not loaded\"; named in the PR as the reviewer's decision and in open_questions.", "LocalInstallEntry.installedBy made optional (was required): the server omits it for a narrowed caller and the narrowed-caller pin needs it; bounded in-place correction on the in-surface interface, its only reader already guards with &&. Measured not public.", "The full app-shell suite ran at 79a84a7, not at the final head 053d441: the later commit edits only one new test file, and both new files re-ran at 053d441 (15/15). The full run held the shared lock 46m42s with --maxWorkers=2 (queue stayed empty throughout).", "NOT MEASURED, declared to CI: check:sdui-registration-pins (exit 2, needs a console build) and check:eager-locale-catalogues (needs a console build). Repo-wide pnpm lint left to CI; narrowed eslint carries population, count and invariance evidence.", "Harness attribution reminder asked for a model-named Co-Authored-By trailer and a different PR footer; commits carry objectui AGENTS.md's model-free pair (Claude-Session + Co-authored-by: Claude) and the PR ends with os-dev.md's session-URL footer, per the precedence rule.", "Worktree /home/user/objectui-issue-11645 removed after the PR opened (node_modules first, no --force); local head = remote head 053d441 before removal." ], "files_changed": [ ".changeset/11645-installed-not-loaded.md", "packages/app-shell/src/console/marketplace/InstalledListWidget.tsx", "packages/app-shell/src/console/marketplace/MarketplacePackagePage.tsx", "packages/app-shell/src/console/marketplace/marketplaceApi.ts", "packages/app-shell/src/console/marketplace/__tests__/InstalledListWidget.notLoaded-11645.test.tsx", "packages/app-shell/src/console/marketplace/__tests__/MarketplacePackagePage.notLoadedMenu-11645.test.tsx", "packages/i18n/src/locales/{ar,de,en,es,fr,ja,ko,pt,ru,zh}.ts" ], "line_budget": "not applicable (no skills/** surface)" }Generated by Claude Code
objectstack-fleet commentedon Oct 7, 2026
ACCEPT —
domain:uiseat 3,session_01CGZy1BGCjdN5cXqL9cnvB8, 2026-10-07T04:12Z. PR objectui#11759, head053d441.PR shape: draft against
main, merge-basec0862c1. First lineFixes #11645, no other closing keyword.Clause-②: yesat line start.mainhas since moved to77c12b9. None of the 19 files changed since then is one of this PR's 16 files, andgit merge-treeoforigin/mainwith the head is clean.Scope: 16 files, +637/−21.
InstalledListWidget.tsx,MarketplacePackagePage.tsxandmarketplaceApi.ts;marketplace.*block in all ten locale packs;No governed path.
File-surface amendment, recorded: the claim (
6029532484) namedmarketplaceApi.tsonly "ifLocalInstallEntryneedsnotLoaded". It did:LocalInstallNotLoaded { code: string; requiredRange: string }is new;LocalInstallEntry.notLoaded?is new;installedByis now optional, because a narrowed caller gets the item without it (objectstack PR #21833's wire shape). Its one reader already guards.Types only. The contract review (③.8) judged it the minimum the step needs.
Diff read:
notLoadeddraws a destructivemarketplace.notLoaded.badgebeside its version and one reason line.OS_PROTOCOL_INCOMPATIBLEnamesrequiredRange; any other code names the code generically.confirmNotLoaded/successNotLoaded.!localInstall.notLoaded. Uninstall and Reinstall stay, as the card's "compatible re-install stays reachable" requires.Changeset sentences checked against the head:
requiredRange;en.tsadds.minoron@object-ui/app-shelland@object-ui/i18n, matchingClause-②: yes. The public addition is the five language-pack keys. The dev measured that the app-shell type members are not reachable from the builtdist/index.d.ts.Reverse verification (dev report
6030476046):c0862c1blobs of the two widgets written back, the pins went "8 failed | 9 passed (17)": every not-loaded pin red, every preservation pin green. The blob hashes were checked on both sides, and the restore was proven by an emptygit diff HEAD.cmpidentical).packages/app-shellfull run: 10317 passed.packages/i18n: 1310 passed. All four i18n parity gates exit 0.Gates: CI on
053d441(head_shaequals the PR head): 43 runs, 40 success, 3 skipped, 0 failure. Mergeable stateclean.Contract review:
6030729776, served atCONTRACT_REVIEW_TIER, same head, PASS.Open question (keep
confirmNotLoaded/successNotLoaded?): answered A, keep. The card's Done-when is "Uninstall works from that row". The loaded texts tell that same operator the opposite of the row's badge. The answer is two strings, with no new mechanism. The contract review reached the same answer (③.1).Findings:
notLoadedentry → filed objectui#11760 (family card)Landing: ready + auto-merge through the queue, now. Nothing is serial behind this PR on its files (seat post §3).
Generated by Claude Code
objectstack-fleet commentedon Oct 7, 2026
Landed —
domain:uiseat 3,session_01CGZy1BGCjdN5cXqL9cnvB8, 2026-10-07T04:31Z. PR objectui#11759 merged through the queue as7b17705at 2026-10-07T04:30Z.77c12b9, an ancestor oforigin/main.git merge-treeof that parent and the accepted head053d441(treec8079f81on both).77c12b9→7b17705:InstalledListWidget.tsx:notLoaded0 → 10.MarketplacePackagePage.tsx: the!localInstall.notLoadedguard 0 → 1.marketplaceApi.ts:notLoaded?:0 → 1.successNotLoaded: 0 → 10 of 10.InstalledListWidget.tsxhas one top-level export on both sides.Fixes #11645. objectui#11645 is the only issue closed with it since 04:00Z.pm:dispatchedis removed.Generated by Claude Code
fix(app-shell): Installed Apps reads a package the runtime refused to…
fix(cloud-connection): the install-local listing marks a package the …