Skip to content

refactor: enforce src/utils retirement (#2149) - #2229

Merged
thymikee merged 1 commit into
mainfrom
refactor/issue-2149-src-utils-retirement
Sep 2, 2026
Merged

refactor: enforce src/utils retirement (#2149)#2229
thymikee merged 1 commit into
mainfrom
refactor/issue-2149-src-utils-retirement

Conversation

@thymikee

@thymikee thymikee commented Sep 2, 2026

Copy link
Copy Markdown
Member

Summary

Enforce the terminal src/utils retirement for #2149:

  • add named R14 tracked-tree enforcement and planted-red coverage;
  • make tracked-path and Fallow metadata truthful after the owner moves;
  • remove stale utility baseline keys and annotate intentional parser fixtures.

Closes #2149.

Validation

  • pnpm check:layering — 190 structural tests passed; guard OK
  • pnpm mutation:test — 40 tests passed
  • pnpm vitest run scripts/__tests__/test-file-size-ratchet.test.ts packages/selectors/src/internal/parse.test.ts — 12 tests passed
  • pnpm check:fallow --base origin/main — no issues in changed files
  • pnpm check:affected --run — all runnable checks passed; GitHub-authoritative/device checks skipped by policy
  • git ls-files -- src/utils — empty

@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown

Size Report

Metric Base Current Diff
JS raw 2.56 MB 2.56 MB 0 B
JS gzip 860.8 kB 860.8 kB 0 B
npm bundled raw 2.56 MB 2.56 MB 0 B
npm bundled gzip 860.8 kB 860.8 kB 0 B
npm tarball 1.01 MB 1.01 MB +11 B
npm unpacked 3.44 MB 3.44 MB 0 B
npm clean-installed 3.44 MB 3.44 MB 0 B

npm unpacked components

Component Base Current Diff
JS / dist source 2.72 MB 2.72 MB 0 B
Apple runner source/project 544.4 kB 544.4 kB 0 B
Apple snapshot presentation source 33.8 kB 33.8 kB 0 B
macOS helper source 54.8 kB 54.8 kB 0 B
Android helper artifacts 43.3 kB 43.3 kB 0 B
Other package files 46.3 kB 46.3 kB 0 B

Startup median (7 runs, lower is better):

Scenario Base Current Diff
CLI --version 32.0 ms 33.6 ms +1.6 ms
CLI --help 80.8 ms 81.5 ms +0.7 ms

Top changed chunks: no changes in the largest emitted chunks.

Top changed packed files

No changed packed files.

@thymikee

thymikee commented Sep 2, 2026

Copy link
Copy Markdown
Member Author

Reviewed exact head 441ee44644d84852bb9337ef5bbbcfaa6a285fd9 against #2149. Clean verdict. All declared owner-slice dependencies are closed; R14 is a real tracked-tree layering gate, not an allowlist: it consumes git ls-files -z -- src/utils, reports every exact/nested path through the shared retired-path owner, and has non-vacuous planted-path coverage including nested/non-TS/Unicode and exact-root cases. The committed tree has zero tracked src/utils paths, retained literals are explicitly arbitrary-path fixtures, and no second registry or packaging growth was introduced. Exact-head CI and all native smoke lanes are green; mergeable and ready for human review.

@thymikee thymikee added the ready-for-human Valid work that needs human implementation, judgment, or maintainer merge label Sep 2, 2026
@thymikee
thymikee merged commit db08548 into main Sep 2, 2026
19 checks passed
@thymikee
thymikee deleted the refactor/issue-2149-src-utils-retirement branch September 2, 2026 05:59
@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-09-02 06:00 UTC

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

Labels

ready-for-human Valid work that needs human implementation, judgment, or maintainer merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor: close src/utils retirement and enforce no-regrowth

1 participant