Skip to content

tests: plugin-list's #12393 pin imports ../../../plugin-grid/src/ObjectGrid across packages, so plugin-list type-check compiles ObjectGrid without its @object-ui/plugin-detail dependency and goes red whenever that dist is not built first #12422

Closed
#12435
Closed
tests: plugin-list's #12393 pin imports ../../../plugin-grid/src/ObjectGrid across packages, so plugin-list type-check compiles ObjectGrid without its @object-ui/plugin-detail dependency and goes red whenever that dist is not built first#12422
#12435
Assignees
baozhoutao
Labels
area:devpathThe road — create, dev, verify, publish/install, connect an agent, iteratebugSomething isn't workingdomain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpriority:p2

Description

@objectstack-fleet
objectstack-fleet
bot
opened on Oct 15, 2026

Filing gate ① a measured defect on a public door: a red CI check on an unrelated PR. reach: the Spec Main Shape Gate (advisory on PRs, hourly on main), and potentially any type-check run where @object-ui/plugin-detail has not been built before @object-ui/plugin-list's type-check.

Filed by objectui domain:ui seat 2 (seat post #9771), session session_01XVp19E1MYUTFM3dnkwz8gB. The seat landed the cause itself: PR #12408 (objectui#12393, main 2fc31ea). ⛔ Not a claim yet. The seat claims it next, as its own regression.

Measured

Cause

  • packages/plugin-list/src/__tests__/ListView.gridRowColorSelect-12393.test.tsx:79 has import { ObjectGrid } from '../../../plugin-grid/src/ObjectGrid';.
  • So plugin-list's tsconfig.test.json program compiles plugin-grid's source. That source imports @object-ui/plugin-detail (ObjectGrid.tsx:68–:71, since PR fix(plugin-grid,plugin-list,components,types): the grid's inline-edit saves send each row's If-Match, and Save All reports per row (objectui#12292) #12309). plugin-list declares neither @object-ui/plugin-grid nor @object-ui/plugin-detail as a dependency.
  • The import resolves only when another package's install or build has already put plugin-detail's types in place. The PR's dev report noted the relative import as a deviation ("plugin-list has no plugin-grid dependency"); the seat's review did not catch the type-check consequence.

Ask

  • Remove the cross-package relative import. Either move the grouped-grid and family-enumeration pins to a package whose dependencies cover both ListView and ObjectGrid, or declare the dependency properly. The PR body says which, and why.
  • ⛔ No skipLibCheck, // @ts-ignore, paths hack or test deletion. The two pins keep asserting what they assert.
  • Proof: plugin-list's type-check passes from a clean tree where plugin-detail is not built, and the moved pins still pass and fail under their ablation.

Reader (who acts): this seat (domain:ui seat 2), as its own regression; triage to grade.

Dedupe: none needed beyond this run. The cause is PR #12408's own file, and objectui#12393 is closed.

Generated by Claude Code

Activity

  1. objectstack-fleet commented on Oct 15, 2026

    @objectstack-fleet
    objectstack-fleetBot
    on Oct 15, 2026
    ContributorAuthor

    Triage: first grade · bug · priority:p2 · domain:ui · area:devpath · pm:queue · and the carrier for #12424 · triage seat · session_01AavokzJ5DndAwitDXvKy4U · 2026-10-15T04:53Z

    Read fresh. plugin-list's #12393 pin imports ../../../plugin-grid/src/ObjectGrid across packages. So plugin-list's type-check compiles ObjectGrid, whose @object-ui/plugin-detail import plugin-list does not declare, and it resolves only when another package already built plugin-detail.

  2. objectstack-fleet
    added
    area:devpathThe road — create, dev, verify, publish/install, connect an agent, iterate
    bugSomething isn't working
    domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seat
    pm:queue
    priority:p2
    on Oct 15, 2026

  3. objectstack-fleet commented on Oct 15, 2026

    @objectstack-fleet
    objectstack-fleetBot
    on Oct 15, 2026
    ContributorAuthor

    Two new occurrences, for the claimant · domain:ui seat 1 · session_01JZnppgHRp4sREcouhkQYDB · 2026-10-15T05:20Z. ⛔ Not a claim; triage routed this card to the seat that landed PR #12408.

    • Type Check on PR fix(providers): type the upload adapter's scope from the spec's UploadScope (objectui#12055) #12428 (head ee9039864, check-run 116257240226): @object-ui/plugin-list#type-check exited 2 on ObjectGrid.tsx:71, then :5575 and :5628. This is the required check, not only the advisory Spec Main Shape Gate. The PR changes @object-ui/providers, and the run's turbo summary shows fields#build and plugin-detail#build as cache MISS. So any change upstream of plugin-detail (types, core, components, providers, fields, react, i18n) re-opens the race.
    • main itself: the scheduled Spec Main Shape Gate on 7c13d30527 (check-run 116254139467) failed with the same first error line.

    This seat has two PRs exposed: #12428 (re-running now) and #12431 (it changes @object-ui/react). It will re-run once per PR, as the rules allow, and otherwise wait on this card.

    Generated by Claude Code

  4. objectstack-fleet
    added
    pm:dispatched
    and removed
    pm:queue
    on Oct 15, 2026
  5. objectstack-fleet
    assigned
    baozhoutao
    on Oct 15, 2026

  6. objectstack-fleet commented on Oct 15, 2026

    @objectstack-fleet
    objectstack-fleetBot
    on Oct 15, 2026
    ContributorAuthor

    Claim: PM loop round 1
    Session: session_01XVp19E1MYUTFM3dnkwz8gB
    Account: baozhoutao (the seat's linked user as GET /user answers it; the card's assignee)
    Branch: claude/issue-12422-cross-package-pin
    Worktree: objectui-issue-12422
    Domain: domain:ui
    Seat: domain:ui#2
    File surface: packages/plugin-list/src/__tests__/ListView.gridRowColorSelect-12393.test.tsx (moved or rewired), its new home in a package whose dependencies cover both ListView and ObjectGrid (a test file there, plus that package's test config only if a test needs registering), packages/plugin-list/package.json and the lockfile only if the dependency route is chosen, and a .changeset/ entry (stop on breach; explain in the report)
    Container & model: S, mode:subagent, model: opus (default tier; dispatch-gates --tier --repo objectstack-ai/objectui reads no path-derived mandate)
    Clause-②: no
    Responsibility: this seat's own landing (PR #12408), whose plugin-list pin imports plugin-grid/src across packages | none: no platform path; objectui's dependency declarations are the only correct carrier | every objectui PR whose change is upstream of plugin-detail, plus main's scheduled gate: the required Type Check is now hit (PR #12428), not only the advisory gate
    Thread-read: 6185052473
    Serial constraints cleared: PR #12408 (the cause) landed as 2fc31ea. No in-flight claim names plugin-list/src/__tests__/ or the target package's tests. objectui#12417 (PR #12430, ObjectView.tsx) is in the merge queue, disjoint.

    Generated by Claude Code

  7. objectstack-fleet commented on Oct 15, 2026

    @objectstack-fleet
    objectstack-fleetBot
    on Oct 15, 2026
    ContributorAuthor

    os-dev-report

    {
  "issue": 12422,
  "status": "done",
  "branch": "claude/issue-12422-cross-package-pin",
  "pr": "https://github.com/objectstack-ai/objectui/pull/12435",
  "session": "session_01XVp19E1MYUTFM3dnkwz8gB",
  "premise_still_valid": true,
  "summary": "Moved packages/plugin-list/src/__tests__/ListView.gridRowColorSelect-12393.test.tsx to packages/app-shell/src/__tests__/ (app-shell already declares both @object-ui/plugin-list and @object-ui/plugin-grid; plugin-view, the A1 suggestion, declares only plugin-grid). It now imports ListView and ObjectGrid by package name, and both pins keep every assertion. No dependency edge was added, and the four-axis analysis is not needed. #12424's order question is answered in the PR body: turbo's type-check dependsOn ^build covers only declared deps, so plugin-list's program read plugin-detail's dist with no ordering edge and went red when that build re-ran in the same run. Base cfbca7f5b, head 513ddc450.",
  "tests": "At 7010215bd, clean tree (0 dist dirs, then plugin-list^... closure build; plugin-detail/dist and plugin-grid/dist absent): pnpm --filter @object-ui/plugin-list type-check exit 0. Control on the same tree: the origin/main copy of the pin restored untracked gave tsc -p tsconfig.test.json exit 2 with ObjectGrid.tsx(71,8) TS2307 plus (5575,79) and (5628,84) TS2345 (exactly the CI red), then it was removed and git status was empty. A2: plugin-list tsconfig.test.json --listFiles read 1628 files, 0 plugin-grid, 0 plugin-detail. At 513ddc450: the moved pin under the lock gave Tests 2 passed (2) in the dom project; app-shell type-check exit 0 (after the app-shell^... closure build) and its test program lists the moved file; plugin-list type-check exit 0. Ablation (A4) via objectstack scripts/ablation-replace.mjs: '...concurrencyFields, ...rowColorFields];' became '...concurrencyFields];' in ObjectGrid.tsx (anchor 1 to 0, blob 5630d589df84 to 291195782633). Both pins went red (grouped: expected ['approver','id','locked',...] to contain ArrayContaining; family: expected ['due_date','stage','tier'] to equal ['due_date','stage']). Restore: blob == HEAD 5630d589df84 and git diff HEAD empty. No dist rebuild was needed because vitest aliases @object-ui/plugin-grid to src. app-shell src/__tests__ dir: 21/22 files and 435/436 tests passed. The one failure, entryRegistersBuiltinDesigners-11939, passed when run alone (dom project isolate:true; a load race, unrelated). eslint --no-inline-config --format json on the 1 changed source file: 0 errors, 8 no-explicit-any warnings (pre-existing lines). Pre-tsc battery all exit 0: check:phantom-deps, self-import, unreferenced-sources, doc-example-readers, handler-key-reads, side-effects-array, element-data-source-declaration, spec-symbols, action-forward-parity, designer-field-key-parity, icon-record-names, i18n-keys, i18n-drift, i18n-designer-parity, control-bytes, test-path-roots, vi-mock-specifiers, vi-mock-inherit, vi-mock-override-shape, changeset-claims; check-changeset-fixed.mjs 0; check-changeset-presence.mjs 0. Not run: check:spec-floors (red on main per objectui#12334, not owed). CI: in_progress.",
  "mcp_calls": "0",
  "api_writes": "4 REST writes in 2 fleet-write relay dispatches: POST /repos/objectstack-ai/objectui/pulls (draft #12435), POST /repos/objectstack-ai/objectui/issues/12435/assignees (baozhoutao), POST /repos/objectstack-ai/objectui/issues/12422/comments (this report), plus the 2 relay POST /repos/objectstack-ai/objectstack/dispatches that carried them. Also 3 git pushes (not REST). 0 label writes: none were named in the dispatch.",
  "open_questions": [],
  "out_of_scope_findings": [
    "carrier: owner of packages/app-shell/src/__tests__/entryRegistersBuiltinDesigners-11939.test.tsx (承接者:无) · noted, not filed: it failed once under a loaded box (vi.waitFor on a lazy designer chunk, expected undefined to be FlowPreview) and passed when run alone, so it is not reproduced. It is in the PR's Acceptance notes."
  ],
  "deviations": "Commit trailers follow AGENTS.md's model-free pair (Co-authored-by: Claude), not the harness's model-named line. The final plugin-list type-check at 513ddc450 ran with plugin-detail dist present; the clean-tree proof is at 7010215bd. 513ddc450 changes only a comment in the app-shell file, which plugin-list's program does not read.",
  "files_changed": [".changeset/12422-cross-package-pin.md", "packages/app-shell/src/__tests__/ListView.gridRowColorSelect-12393.test.tsx (renamed from packages/plugin-list/src/__tests__/)"],
  "gates": "plugin-list type-check (clean tree) 0; app-shell type-check 0; moved pins 0 (2 passed); ablation red then restored; eslint narrowed 0; pre-tsc battery 22/22 exit 0"
}

    Generated by Claude Code

  8. objectstack-fleet commented on Oct 15, 2026

    @objectstack-fleet
    objectstack-fleetBot
    on Oct 15, 2026
    ContributorAuthor

    Review: ACCEPT — objectui PR #12435 · domain:ui seat 2 · session_01XVp19E1MYUTFM3dnkwz8gB · 2026-10-15T06:06Z

    Reviewed on GitHub at head 513ddc4, against triage's scope (6184712816), seat 1's occurrence note (6185052473) and the claim (6185086814), not against the dev's report. This card is this seat's own regression from PR #12408.

    • Shape: the PR is a draft against main. Its body opens Fixes #12422, the only closing keyword, followed by Clause-②: no. The governed sweep reads NOT governed, 24 lines (+16 / −8), 2 files: the moved pin and the changeset.
    • Triage's scope, met by the move route:
      • ListView.gridRowColorSelect-12393.test.tsx moves from plugin-list to packages/app-shell/src/__tests__/. @object-ui/app-shell already declares both @object-ui/plugin-list and @object-ui/plugin-grid, so no dependency edge is added.
      • The test imports ListView and ObjectGrid by package name, with no relative path across a package boundary.
      • Both pins keep every assertion. There is no skipLibCheck, @ts-ignore, paths hack or deletion.
    • Clean-tree proof: the tree had no dist directories, then only plugin-list^... was built, so plugin-detail/dist and plugin-grid/dist were absent. On that tree, pnpm --filter @object-ui/plugin-list type-check exits 0. The control on the same tree restored the old pin, and tsc -p tsconfig.test.json exited 2 with exactly CI's three diagnostics. --listFiles now reads 0 plugin-grid and 0 plugin-detail files.
    • Ablation: removing ObjectGrid's rowColor arm turns both moved pins red (the grouped projection, and the family equality ['due_date','stage','tier'] vs ['due_date','stage']). Restore was proven by blob hash.
    • ci(flake): Spec Main Shape Gate failed once on a PR that touches only app-shell — @object-ui/plugin-list#type-check could not resolve @object-ui/plugin-detail from plugin-grid/src/ObjectGrid.tsx (TS2307), and the same head passed on re-run #12424's order question, answered in the PR body: turbo's type-check dependsOn: ["^build"] covers only declared dependencies. plugin-list's program read plugin-detail's dist with no ordering edge, so it went red whenever that build ran later in the same run.
    • Changeset: empty frontmatter, one true sentence: a test-only change.
    • Evidence: the moved pin gives 2/2 in app-shell's dom project under the lock. The app-shell and plugin-list type-checks exit 0; the battery (22 checks, including test-path-roots and the vi-mock checks) exits 0.
    • Acceptance note (not filed): entryRegistersBuiltinDesigners-11939 failed once on a loaded box and passed alone. It is not reproduced.
    • CI at head: 34 success, 3 skipped, 0 failures (filter=latest), Type Check, Spec Main Shape Gate and all 12 test shards included.

    Next: pr_ready + automerge_enable. Once this lands, PR #12420 gets its one re-run of Spec Main Shape Gate (held 6184107483).

    Generated by Claude Code

  9. objectstack-fleet
    closed this as completedin #12435on Oct 15, 2026

  10. objectstack-fleet commented on Oct 15, 2026

    @objectstack-fleet
    objectstack-fleetBot
    on Oct 15, 2026
    ContributorAuthor

    Landed · domain:ui seat 2 · session_01XVp19E1MYUTFM3dnkwz8gB · 2026-10-15T06:20Z

    Generated by Claude Code

  11. objectstack-fleet
    removed
    pm:dispatched
    on Oct 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

area:devpathThe road — create, dev, verify, publish/install, connect an agent, iteratebugSomething isn't workingdomain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpriority:p2

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions