Skip to content

refactor: delete capabilities projection shell - #2094

Merged
thymikee merged 3 commits into
mainfrom
refactor/remove-capabilities-shell
Aug 27, 2026
Merged

thymikee merged 3 commits into
mainfrom
refactor/remove-capabilities-shell

Conversation

@thymikee

Copy link
Copy Markdown
Member

Summary

Move the runtime-use and public runtime-fact projections into the descriptor registry and delete the residual capabilities shell. This preserves descriptor-derived admission, action-selected use semantics, and public-command filtering without a compatibility re-export.

Validation

Focused descriptor/inventory/eager-closure tests: 340 passed.
Layering: 150 passed.
pnpm check:affected --run completed all 3,410 tests on the final retry; Vitest reported existing parallel-load teardown errors, while the affected suites pass in isolation.
GitHub-authoritative device lanes remain for CI.

@github-actions

github-actions Bot commented Aug 27, 2026 •

Copy link
Copy Markdown

Size Report

Metric Base Current Diff
JS raw 2.48 MB 2.48 MB +25 B
JS gzip 834.0 kB 834.0 kB +1 B
npm tarball 957.2 kB 957.2 kB +11 B
npm unpacked 3.32 MB 3.32 MB +25 B

npm unpacked components

Component Base Current Diff
JS / dist source 2.63 MB 2.63 MB +25 B
Apple runner source/project 581.1 kB 581.1 kB 0 B
macOS helper source 54.8 kB 54.8 kB 0 B
Android helper artifacts 0 B 0 B 0 B
Other package files 45.3 kB 45.3 kB 0 B

Startup median (7 runs, lower is better):

Scenario Base Current Diff
CLI --version 23.9 ms 24.9 ms +1.0 ms
CLI --help 63.4 ms 67.2 ms +3.8 ms

Top changed chunks:

Chunk Raw diff Gzip diff
dist/src/session2.js -268 B -105 B
dist/src/sdk-batch-runner.js +291 B +77 B
dist/src/app-inventory-contract.js +2 B +14 B
dist/src/perf-runtime-plan.js +2 B -8 B
dist/src/runtime4.js 0 B -6 B

Top changed packed files

Packed file Base Current Diff
dist/src/sdk-batch-runner.js 80.6 kB 80.9 kB +291 B
dist/src/session2.js 216.6 kB 216.3 kB -268 B
dist/src/runtime.js 64.0 kB 64.0 kB -7 B
dist/src/internal/daemon.js 110.0 kB 110.0 kB +5 B
dist/src/app-inventory-contract.js 44.2 kB 44.2 kB +2 B
dist/src/perf-runtime-plan.js 63.6 kB 63.6 kB +2 B

@thymikee

Copy link
Copy Markdown
Member Author

[P2] Current main now rejects implementation comments that preserve review history or narrate control flow, but the edited gesture-facts.ts header still carries R52/R53 and retired core/capabilities provenance. Remove that narrative; let the fact table, symbols, and tests express the invariant, with history remaining in the ADR/PR.

The descriptor-projection move itself is sound, but this head is not ready while Integration Tests is red on daemon SIGTERM cleanup and iOS Smoke is pending.

@thymikee

Copy link
Copy Markdown
Member Author

Addressed the P2 in c09f421 by removing the R52/R53 and retired-capability provenance header from packages/platform-apple/src/gesture-facts.ts. The fact table and behavior are unchanged. The new commit is pushed; CI has restarted. The prior Integration Tests failure was the unrelated live web smoke assertion that the daemon exited after SIGTERM.

@thymikee

thymikee commented Aug 27, 2026 •

Copy link
Copy Markdown
Member Author

The capabilities-shell deletion is functionally clean: both projections now read unchanged semantics directly from the descriptor registry. Not ready yet: the gesture-facts follow-up removed only its header; delete the remaining retired-admission/control-flow narration, especially lines 51-55, 62-65, and 98-101. Exact-head iOS is also red on TEXT_INPUT_COMMIT_NOT_OBSERVED; apparently unrelated, but it must rerun green before readiness.

@thymikee

Copy link
Copy Markdown
Member Author

Addressed this comment in 24cac91: removed the remaining private-helper admission/control-flow/history narration from packages/platform-apple/src/gesture-facts.ts, including the cited ranges. The exported projection documentation and fact behavior are unchanged. The local affected gate passed (415 files, 3,410 tests); exact-head GitHub checks are now rerunning, including iOS Smoke.

@thymikee

Copy link
Copy Markdown
Member Author

Code review is clean at 24cac913a6: the delta removes all previously cited implementation/history narration without changing semantics. All non-iOS checks are green. Exact-head iOS still fails on the gesture-pan replay; current main is also red on a different iOS scenario, so this appears lane instability rather than a code finding, but require a green rerun or same-load base comparison before merge readiness.

@thymikee
thymikee merged commit 77b5be8 into main Aug 27, 2026
18 of 19 checks passed
@thymikee
thymikee deleted the refactor/remove-capabilities-shell branch August 27, 2026 18:10
@github-actions

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-08-27 18:14 UTC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant