Conversation
…same pattern as #1377; second-lowest-risk)
There was a problem hiding this comment.
Pull request overview
This PR migrates the memory-reference-existence-lint workflow from the legacy shell implementation to the Bun/TypeScript port, following the same rollout pattern as the earlier duplicate-index lint migration. It keeps the workflow aligned with the repository’s TS+Bun hygiene-tool standardization while adding the toolchain bootstrap needed on CI.
Changes:
- Swaps the workflow’s referenced lint tool from
audit-memory-references.shtoaudit-memory-references.ts. - Adds a
tools/setup/install.shstep so Bun is available on the runner before invoking the TypeScript tool. - Updates workflow path filters and inline comments to point at the
.tsimplementation.
…flow Reviewer-caught: FACTORY-HYGIENE.md row 59 documented `tools/hygiene/audit-memory-references.sh --enforce` as the shipped tool, but PR #1378 switches the workflow to call `bun tools/hygiene/audit-memory-references.ts --enforce`. The doc and the workflow would have disagreed the moment #1378 merged. Two changes in row 59: - Tool invocation: `tools/hygiene/audit-memory-references.sh --enforce` → `bun tools/hygiene/audit-memory-references.ts --enforce` - Composes-with pointer: `tools/hygiene/audit-memory-references.sh` → `tools/hygiene/audit-memory-references.ts` Composes with #1377 (memory-index-duplicate-lint.yml .sh→.ts), which landed without a corresponding FACTORY-HYGIENE row update because no row documents that workflow yet (the audit-memory-index-duplicates.ts tool was never called out by row in the same way row 59 calls out audit-memory-references). Future TS-port work should grep FACTORY-HYGIENE.md for the .sh pointer before opening the workflow PR.
3 tasks
3 tasks
AceHack
added a commit
that referenced
this pull request
May 3, 2026
…t tier per #1376) (#1380) The gate.yml workflow ran three lint scripts as .sh: - tools/hygiene/check-tick-history-order.sh - tools/hygiene/check-no-conflict-markers.sh - tools/hygiene/check-archive-header-section33.sh All three have a TS+Bun port in tools/hygiene/<name>.ts with full output / exit-code parity verified locally (179 tick-history rows non-decreasing; 0 conflict-marker violations; 0 §33 archive-header violations). Why this PR closes the .sh→.ts cleanup loop: - #1377 converted memory-index-duplicate-lint.yml (lowest-risk — only fires on memory/MEMORY.md changes) - #1378 converted memory-reference-existence-lint.yml (lowest-risk — only fires on memory/** changes) - This PR converts gate.yml's three lint jobs (highest-risk — fires on every PR via the all-PR gate critical path) - #1379 (rerere ALLOWLIST parity) was a prerequisite — the .ts port of check-no-conflict-markers had to ALLOWLIST the rerere memo to match the .sh version BEFORE running on every PR After this lands and CI verifies green, the 5 deferred .sh files can be removed (audit-memory-index-duplicates.sh, audit-memory-references .sh, check-archive-header-section33.sh, check-no-conflict-markers.sh, check-tick-history-order.sh) — completing the .sh cleanup the maintainer asked for in #1371 ("we should clean up .sh and any .sh where we have the .ts I thought we were fully converted"). Conversion pattern (identical for all three jobs): - Add "Install toolchain via three-way-parity script" step calling ./tools/setup/install.sh — same shape as #1377 + #1378 - Change `run: tools/hygiene/<name>.sh` to `run: bun tools/hygiene/<name>.ts` Three lint jobs run in parallel (one job each), so the bun install cost is bounded by the slowest job's path — does not compound.
AceHack
added a commit
that referenced
this pull request
May 3, 2026
…rsion (#1380) + #1378 reviewer fix (#1381) Final risk-strat tier of the .sh→.ts CI cleanup. PR #1380 opens with all 3 gate.yml lint jobs (check-tick-history-order, check-no-conflict-markers, check-archive-header-section33) converted in one bounded PR — rationale: independent parallel jobs, parity verified locally, mitigates 3-PR round-trip cost. PR #1378 reviewer thread (FACTORY-HYGIENE.md row 59) addressed in commit 3e0342c on that branch. Thread resolved via GraphQL. Discipline lesson: pre-flight surface scan as parity-debt prevention.
3 tasks
AceHack
added a commit
that referenced
this pull request
May 3, 2026
…conversion completed (#1382) The maintainer asked in #1371: *"we should clean up .sh and any .sh where we have the .ts I thought we were fully converted."* Initially 13 zero-ref .sh files were removed; 5 were deferred because they were still referenced from CI workflows. Those 3 workflows are now all converted (#1377 + #1378 + #1380), and #1379 fixed the prerequisite allowlist parity bug in check-no-conflict-markers.ts. Removed: - tools/hygiene/audit-memory-index-duplicates.sh (→ memory-index-duplicate-lint.yml via #1377) - tools/hygiene/audit-memory-references.sh (→ memory-reference-existence-lint.yml via #1378) - tools/hygiene/check-archive-header-section33.sh (→ gate.yml via #1380) - tools/hygiene/check-no-conflict-markers.sh (→ gate.yml via #1380) - tools/hygiene/check-tick-history-order.sh (→ gate.yml via #1380) All 5 .ts ports run clean locally post-deletion (exit 0; same output as the .sh equivalents — verified manually before this commit). Surface-update set (active substrate that pointed at the .sh as a current tool): - tools/hygiene/LOST-FILES-LOCATIONS.md — survey command + composes-with pointers updated from .sh to .ts (the only runtime-bound reference) - docs/trajectories/typescript-bun-migration/RESUME.md — Bucket D listing updated; 5 entries removed; new "Removed 2026-05-03" note added explaining why - tools/hygiene/audit-orphan-role-refs.sh — comment composes-with template updated to .ts - tools/hygiene/check-role-ref-on-current-state-surfaces.sh — comment composes-with sibling-pattern updated to .ts - tools/hygiene/sort-tick-history-canonical.ts — comment composes-with detection-check updated to .ts - tools/hygiene/check-tick-history-shard-schema.sh — comment composes-with legacy-monolithic-check updated to .ts - tools/lint/doc-comment-history-audit.baseline — 2 entries for the deleted audit-memory-references.sh removed (else baseline would diff against the next audit run that no longer matches a deleted path) Historical surfaces deliberately NOT updated (pure history record): - docs/hygiene-history/** (tick shards + loop-tick-history.md) - docs/pr-preservation/** (preservation logs) - docs/research/** (research reports) - memory/** (history-surface attribution carve-out) - docs/backlog/** rows (describe past tickets that referenced .sh) - docs/0-0-0-readiness/CLASSIFICATION.md (sync-classification audit of past LFG↔AceHack divergence) Closes the .sh→.ts cleanup loop opened in #1371. Composes with #1373 + #1374 (stale-ref sweeps that caught earlier comprehensive-ref-check misses).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Same pattern as #1377 (memory-index-duplicate-lint). Second-lowest-risk per #1376 stratification. CLI flag parity verified locally (--enforce; both produce 'memory-reference audit on memory/MEMORY.md ... refs checked: 788'; exit 0). Adds install.sh setup step + bun .ts call. .sh kept during transition.