cloud-connection: the declared-rename planner service is registered only by the install door's plugin, so a kernel without the door cannot reach it #31895

Activity

1. objectstack-fleet

   added

   enhancementNew feature or requestNew feature or request

   priority:p2Medium: important, M3Medium: important, M3

   on Nov 9, 2026
2. objectstack-fleet commented on Nov 9, 2026

   

   objectstack-fleetBot

   on Nov 9, 2026

   ContributorAuthor

   More actions

   triage#1 · os-zhuang · session_01AavokzJ5DndAwitDXvKy4U

   Path: none

   Triage: domain:server · p2 · pm:queue · 2026-11-09T10:54Z

   ⛔ Not a claim.

   • Lane: domain:server, packages/cloud-connection. The declared-rename planner service is registered outside the install door's plugin, so a kernel without that door can still read it. Per [Decision] repo:cloud is used in two senses: 6 open cards are the cloud PM's whole (with domain:cloud), 20 are objectstack-lane work that cloud asked for. Which one does the label mean? #31154 A, no repo:cloud. It follows cloud-connection: expose the declared-rename install planner (planInstallDeclaredRenames) to a hosted control plane reading, through a kernel service #31801.
   • Grade: p2 as filed.

   ---

   Generated by Claude Code
3. objectstack-fleet

   added

   domain:serverDomain lane — seat card indexed by label:pm:seatDomain lane — seat card indexed by label:pm:seat

   pm:queueReady for the PM dispatch loopReady for the PM dispatch loop

   on Nov 9, 2026
4. objectstack-fleet commented on Nov 9, 2026

   

   objectstack-fleetBot

   on Nov 9, 2026

   ContributorAuthor

   More actions

   domain:server#3 · os-steve · session_01ADHkmQXLfvyB5R53VDm3Ka

   Claim: PM loop round 12 · #31895 · 2026-11-09T11:42Z
   Session: session_01ADHkmQXLfvyB5R53VDm3Ka (seat); dev = card thread session_01ADHkmRtcdMB6D762rioHpq
   Account: os-steve
   Branch: claude/project-thread-pxdf8z
   Worktree: the card thread's checkout of objectstack-ai/objectstack
   Domain: domain:server
   Seat: domain:server#3
   File surface:

   • packages/cloud-connection/src/**: the planner moves to its own module, plus a new planner-only plugin and its root export;
   • its tests;
   • .changeset/31895-*.md.
     Stop on breach.
     Container & model: S–M, mode:thread, model: default tier, effort: medium
     Clause-②: no. This adds one export, MarketplaceInstallRenamePlannerPlugin. The planner signature, its result types and the service name are unchanged. If step 0 shows that an existing export or service shape must change, the seat will re-state this line as yes.
     Metadata-protocol: none
     Responsibility: n/a. This follows cloud-connection: expose the declared-rename install planner (planInstallDeclaredRenames) to a hosted control plane reading, through a kernel service #31801, and cloud#3862 is the consumer.
     Thread-read: none
     Premise: step 0 re-measures it on main with a real kernel without the install door, where getService should throw. If main already offers a mount without the door, the card goes back to the seat to be closed.
     Narrowing: no.
     CONTROL:
   • a kernel with only the install door keeps the service;
   • the existing declared-rename suites stay green and are left unedited;
   • with both plugins mounted, in either order, the service is registered once.
     Serial constraints cleared: cloud-connection: expose the declared-rename install planner (planInstallDeclaredRenames) to a hosted control plane reading, through a kernel service #31801's PR feat(cloud-connection): the install door's declared-rename planner is reachable read-only on a kernel service #31851 has merged. No open PR touches marketplace-install-local-plugin.ts.
     Ruling: the seat's delta, 31895-delta.md:
   • a new plugin that registers only the service;
   • one shared implementation module;
   • not on the manifest service.
5. objectstack-fleet

   added

   pm:dispatchedDispatched to a dev agent by /pm-dispatchDispatched to a dev agent by /pm-dispatch

   and removed

   pm:queueReady for the PM dispatch loopReady for the PM dispatch loop

   on Nov 9, 2026
6. objectstack-fleet

   assigned

   os-steve

   on Nov 9, 2026
7. objectstack-fleet commented on Nov 9, 2026

   

   objectstack-fleetBot

   on Nov 9, 2026

   ContributorAuthor

   More actions

   domain:server#3 · os-steve · session_01ADHkmRtcdMB6D762rioHpq
   os-dev-report

   {
    "issue": 31895,
    "status": "done",
    "branch": "claude/project-thread-pxdf8z",
    "pr": "https://github.com/objectstack-ai/objectstack/pull/31929",
    "session": "session_01ADHkmRtcdMB6D762rioHpq",
    "premise_still_valid": true,
    "summary": "New MarketplaceInstallRenamePlannerPlugin (root export of @objectstack/cloud-connection) registers only marketplace-install-rename-planner: no route, no ledger, no rehydrate; objectql is read at plan() time. planDeclaredRenames, convertPackageBodyForward, packageUpgradeSideOf and the service object moved to marketplace-install-rename-planner.ts; the install door registers through the same registerInstallRenamePlanner, which skips when ctx.getServices() already holds the name, so both mounts work in either order. Planner signature, types, service name and the door's providesServices unchanged. Step 0 on origin/main 6a8f4151: door-less ObjectKernel+ObjectQLPlugin getService throws [Kernel] Service 'marketplace-install-rename-planner' not found; CONTROL with door: present; two init() registrations of one name: bootstrap rejects 'already registered', kernel stopped; no door-less mount on main. Cloud mount: await kernel.use(new ObjectQLPlugin()); await kernel.use(new MarketplaceInstallRenamePlannerPlugin());",
    "tests": "head f9562c4a. declared-rename harness: 27 passed (21 existing unedited + 6 new in one new top-level describe). pnpm --filter @objectstack/cloud-connection test: Test Files 80/80, Tests 1144/1144, VERDICT command-exit 0. typecheck (both programs, closure built with dts): VERDICT command-exit 0. Ablations via scripts/ablation-replace.mjs (src imports, no dist leg), each restored blob==HEAD, git diff HEAD empty: A plugin registers nothing -> 3 failed/24 passed; B presence check removed -> both-order pins red with 'already registered' (2 failed/25); C plugin also registers marketplace-install-local -> absence pins red (4 failed/23).",
    "gates": "dispatch-gates --commands at f9562c4a: 69 families, all exit 0 (check-plugin-teardown-shape --self-test first exited 3 on the shallow clone, 0 after fetching fixture commit 621a4876; check:dual-build-cjs-loads first exited 3 PREREQUISITE NOT MET, 0 after full packages build). --ran: 69 derived, 69 run, 0 NOT-MEASURED, 0 UNRUN (derived zero). pnpm lint: package has no lint script (ERR_PNPM_RECURSIVE_EXEC_FIRST_FAIL exit 254); eslint --no-inline-config on the 4 touched source files exit 0. CI check-runs: in_progress at report time.",
    "dogfood_battery": "n/a — no `!` in the PR title and no (narrowing) arm",
    "mcp_calls": "4 — issue_read get, issue_read get_comments x2, pull_request_read get; no write tool",
    "api_writes": "2 — relay pr_create (PR #31929, 8902 bytes sent == stored), post-stamped os-dev-report comment on #31895",
    "open_questions": [
     {
      "question": "Clause-② on a new root export: claim says `no`; #31801 graded its new root exports `yes (widening)`.",
      "options": [
       "A keep `no` as claimed",
       "B seat restates `yes (widening)` on the PR body and changeset"
      ],
      "recommendation": "B if the seat follows #31801's precedent; copied as claimed meanwhile."
     },
     {
      "question": "New public names: plugin class MarketplaceInstallRenamePlannerPlugin (seat-named) and plugin name com.objectstack.runtime.marketplace-install-rename-planner (mirrors the door's com.objectstack.runtime.marketplace-install-local).",
      "options": [
       "A keep",
       "B rename"
      ],
      "recommendation": "A"
     },
     {
      "question": "The plugin declares no dependencies/optionalDependencies on ObjectQLPlugin, since init reads nothing and plan() resolves objectql lazily (no engine -> the door's own transport refusal).",
      "options": [
       "A keep lazy, no declaration",
       "B hard dependency on com.objectstack.engine.objectql"
      ],
      "recommendation": "A: a hard dependency would refuse a boot the planner can still answer, and the door declares none either."
     }
    ],
    "security_findings": "none",
    "out_of_scope_findings": [],
    "files_changed": [
     ".changeset/31895-install-rename-planner-plugin.md",
     "packages/cloud-connection/src/index.ts",
     "packages/cloud-connection/src/marketplace-install-rename-planner.ts",
     "packages/cloud-connection/src/marketplace-install-local-plugin.ts",
     "packages/cloud-connection/src/marketplace-install-local-declared-rename.test.ts"
    ],
    "line_budget": "+305 / -57 over 5 files (dispatch-gates count), under the 3000 human-merge threshold",
    "deviations": [
     "changeset patch instead of the delta's minor: check-changeset-no-major refuses minor beside Metadata-protocol: none (AGENTS.md: level follows the metadata protocol alone)",
     "origin/main moved 6 commits after branch point (6a8f4151 -> 17606e46), none in cloud-connection or the planner's imports; not merged before the PR",
     "PR assignee/reviewer not set: the session's platform asked for MCP issue_write/update_pull_request, refused by standard order rule 3 and os-dev rule 2",
     "no PR-activity subscription: os-dev contract overrides the platform's subscribe posture; CI convergence is the seat's"
    ]
   }

   Stamped 2026-11-09T12:30Z.

   ---

   Generated by Claude Code
8. objectstack-fleet commented on Nov 9, 2026

   

   objectstack-fleetBot

   on Nov 9, 2026

   ContributorAuthor

   More actions

   domain:server#3 · os-steve · session_01ADHkmQXLfvyB5R53VDm3Ka

   Claim amendment · #31895 · 2026-11-09T12:31Z · amends Claim 6739196106
   Clause-②: yes (widening). The package root gains MarketplaceInstallRenamePlannerPlugin and the registerInstallRenamePlanner seam. This matches how #31801 recorded its new root export. Nothing existing narrows: the planner signature, its result types, the service name and the door's providesServices are unchanged. Metadata-protocol stays none, so the changeset stays patch (maintainer ruling A on #25105).
9. objectstack-fleet

   closed this as completedin #31929on Nov 9, 2026
10. objectstack-fleet commented on Nov 9, 2026

    

    objectstack-fleetBot

    on Nov 9, 2026

    ContributorAuthor

    More actions

    domain:server#3 · os-steve · session_01ADHkmQXLfvyB5R53VDm3Ka

    Landed · #31895 · 2026-11-09T13:33Z

    Reading	Value
    PR	#31929, merged 2026-11-09T13:33Z through the queue as merge commit e0c2720c, from head d0545986 (seat ACCEPT 6740577589).
    Card	closed completed by Fixes #31895

    pm:dispatched is removed in this stroke. MarketplaceInstallRenamePlannerPlugin, exported from the @objectstack/cloud-connection root, registers only marketplace-install-rename-planner, so a kernel without the install door can plan declared renames. The door and the plugin share one implementation, and mounting both, in either order, registers the service once.

    For cloud#3862, at its next framework pin: await kernel.use(new ObjectQLPlugin()); await kernel.use(new MarketplaceInstallRenamePlannerPlugin());
11. objectstack-fleet

    removed

    pm:dispatchedDispatched to a dev agent by /pm-dispatchDispatched to a dev agent by /pm-dispatch

    on Nov 9, 2026
12. rajeev-kumar-se

    added a commit that references this issue on Nov 10, 2026

    feat(cloud-connection): mount the declared-rename planner without the…

    ...

    This issue will close once commit e0c2720 is merged into the 'main' branch.

    e0c2720