Skip to content

chore(ui-tooling): drop two test-only scripts and six CLI aliases nothing runs (slice 2 of #25920) - #26024

Merged
objectstack-fleet[bot] merged 1 commit into
mainfrom
claude/project-thread-slsqxn
Oct 26, 2026
Merged

objectstack-fleet[bot] merged 1 commit into
mainfrom
claude/project-thread-slsqxn

Conversation

@objectstack-fleet

Copy link
Copy Markdown
Contributor

Requested by Elon · project thread

Part of #25920 — slice 2 of the unwired-tooling cleanup (slice 1 was #26004). The card stays open for the slices listed at the end.

Summary

Deletions only: 7 files, −1,322 lines, on main 88531fa.

  • ui/scripts/coverage-red-cause-census.mjs + coverage-red-cause-suspects.json + its test → delete. Its header calls it "a ONE-OFF CENSUS, not a gate … nothing in CI calls it"; nothing but its test and one markdown-test-inputs.mjs row named it (row removed).
  • ui/scripts/extract-mdx-demos.mjs + its test → delete. A one-off MDX-to-schema-catalog migration helper; no alias, no workflow; nothing but its test and one markdown-test-inputs.mjs row named it (row removed).
  • ui/package.json aliases cli, objectui, doctor, studio, generate, g → delete. All six point at packages/cli/dist/cli.js; no workflow, script or document runs any of them (exact pnpm NAME / pnpm run NAME grep over the whole tree: 0 hits; the objectui:build / objectui:refresh hits are different root scripts).

Corrections to the #26004 body's "remaining slices" table, kept out of this PR on purpose:

  • test:unit and test:integration stay: ui/scripts/__tests__/package-scripts-vitest-projects.test.ts pins both against the vitest projects.
  • start and site:start stay: both targets exist (@object-ui/console preview, @object-ui/site start) and ui/apps/console/README.md documents pnpm start.
  • check stays: root ci.yml runs pnpm --dir ui run check.

Nothing is wired.

Verification (head e1651b4)

  • Exact-name git grep across the whole tree (AGENTS.md, skills, .claude, docs, every manifest, CHANGELOGs excluded) for both scripts, the ledger and the six aliases: no hits outside the deleted files and the two removed rows. No governed text touched.
  • ui suites that read the scripts inventory and manifest (markdown-test-inputs, package-scripts-vitest-projects, check-unreferenced-sources, ci-cd-pipeline-doc, vitest-invocation-guard): 5 files, 200 tests passed. pnpm --dir ui run check:unreferenced-sources exit 0, check:test-path-roots exit 0.
  • dispatch-gates --commands derived families: see the report on the card for the per-family exit codes.
  • No changeset: private ui tooling and the private ui root manifest only.

Remaining slices


Generated by Claude Code

🤖 Generated with Claude Code

https://claude.ai/code/session_012eXtKYmXEnCc1GExP4go12

…hing runs

- ui/scripts/coverage-red-cause-census.mjs, its suspects ledger and test:
  a one-off census whose own header says no workflow runs it; only its test
  and one markdown-test-inputs row named it.
- ui/scripts/extract-mdx-demos.mjs and its test: a one-off MDX-to-catalog
  migration helper; only its test and one markdown-test-inputs row named it.
- ui/package.json: cli, objectui, doctor, studio, generate and g pointed at
  packages/cli/dist/cli.js; no workflow, script or document invokes them.
  `check` stays (root ci.yml runs it); start, site:start, test:unit and
  test:integration stay (live targets, pinned or documented).

Co-authored-by: Claude <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012eXtKYmXEnCc1GExP4go12
@objectstack-fleet objectstack-fleet Bot added the skip-changeset PR has no user-facing published change; bypasses the changeset gate label Oct 26, 2026
@github-actions github-actions Bot added size/xl dependencies Pull requests that update a dependency file tests labels Oct 26, 2026
@objectstack-fleet

Copy link
Copy Markdown
Contributor Author

席位核对 — ACCEPT at e1651b46

Seat review of #26024, slice 2 of #25920 (devx seat #6023).

  • Deletions only: 7 files, −1,322 lines, not governed (check-governed-merges --pr 26024: 0 of 7 paths on the register), under the 3,000-line human-merge threshold.
  • Each deletion has its one-line reason in the body. coverage-red-cause-census self-describes as a one-off census nothing in CI calls; extract-mdx-demos is a finished migration helper. Their only other reader was a markdown-test-inputs.mjs row each, and both rows go in this PR.
  • The six ui/package.json aliases (cli, objectui, doctor, studio, generate, g): an exact pnpm [run] NAME grep over the PR head finds no caller. The one hit is a historical root CHANGELOG line, which is not a caller.
  • The three corrections to chore(ui-tooling): drop the dead turbo globalDependencies marker and three scripts nothing runs (slice 1 of #25920) #26004's table (test:unit/test:integration, start/site:start, check stay) match the pins and callers the body names.
  • skip-changeset is right: private ui tooling and the private ui root manifest only.
  • CI at e1651b46: 47 success, 13 skipped, 0 failed.
  • Contract review not triggered: no packages/spec/src/**, no governed rule text.

Out of scope, noted by the thread: AGENTS.md and the checklist-test / dogfood-verification skills still name pnpm objectui:build / objectui:refresh, which the root manifest no longer has. That is governed text, so it needs its own card.

Landing: pr_ready + automerge_enable now, under the maintainer's standing grant. #25920 stays open for slice 3 onward.

Card: #25920


Generated by Claude Code

@objectstack-fleet
objectstack-fleet Bot marked this pull request as ready for review October 26, 2026 09:46
@objectstack-fleet
objectstack-fleet Bot enabled auto-merge October 26, 2026 09:46
@objectstack-fleet
objectstack-fleet Bot added this pull request to the merge queue Oct 26, 2026
Merged via the queue into main with commit bbf7061 Oct 26, 2026
61 checks passed
@objectstack-fleet
objectstack-fleet Bot deleted the claude/project-thread-slsqxn branch October 26, 2026 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file size/xl skip-changeset PR has no user-facing published change; bypasses the changeset gate tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants