Conversation
… follow-through on Aaron 'fully converted' directive) PR #1371 removed 13 .sh files in tools/hygiene/. Six memory files referenced 5 of them. This PR mechanically replaces .sh with .ts in those memos so future-Otto's grep for the tool name resolves to the file that exists. Files updated: - memory/feedback_skills_as_carved_sentences_knowledge_in_docs_datavault_2_0_pattern_aaron_2026_05_03.md - memory/feedback_cross_platform_parity_hygiene_deferred_enforcement.md - memory/feedback_enforcing_intentional_decisions_not_correctness.md - memory/feedback_otto_342_heartbeat_row_is_existence_marker_aaron_existential_framing_the_system_is_you_is_your_life_worth_the_cost_2026_04_26.md - memory/project_memory_md_over_cap_2_4x_drift_surfaced_by_snapshot_tool_compaction_candidate_2026_04_23.md - memory/feedback_decision_audits_for_everything_that_makes_sense_mini_adr.md Replacements (5 tool names, mechanical sed): - tools/hygiene/audit-cross-platform-parity.sh → .ts - tools/hygiene/audit-missing-prevention-layers.sh → .ts - tools/hygiene/audit-tick-history-bounded-growth.sh → .ts - tools/hygiene/capture-tick-snapshot.sh → .ts - tools/hygiene/check-no-op-cadence-pattern.sh → .ts Decision-graph edge preservation per B-0141 (brittle-pointer rewriter spec): tool-rename creates broken edges in memory; mechanical rewrite restores edges. Cleanup completes Aaron's "fully converted" directive at the cross-reference layer.
There was a problem hiding this comment.
Pull request overview
Updates stale cross-references inside memory/** files after PR #1371 removed several tools/hygiene/*.sh scripts, keeping memo→tool links accurate by switching those references to the corresponding .ts ports.
Changes:
- Replace
tools/hygiene/*.shreferences withtools/hygiene/*.tsin 6 memory files. - Restore valid cross-reference edges to the current hygiene tool implementations post-cleanup.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| memory/project_memory_md_over_cap_2_4x_drift_surfaced_by_snapshot_tool_compaction_candidate_2026_04_23.md | Updates snapshot tool reference from .sh to .ts. |
| memory/feedback_skills_as_carved_sentences_knowledge_in_docs_datavault_2_0_pattern_aaron_2026_05_03.md | Updates no-op cadence pattern tool reference from .sh to .ts. |
| memory/feedback_otto_342_heartbeat_row_is_existence_marker_aaron_existential_framing_the_system_is_you_is_your_life_worth_the_cost_2026_04_26.md | Updates bounded-growth audit reference from .sh to .ts (two occurrences). |
| memory/feedback_enforcing_intentional_decisions_not_correctness.md | Updates missing-prevention-layers audit reference from .sh to .ts. |
| memory/feedback_decision_audits_for_everything_that_makes_sense_mini_adr.md | Updates parity audit reference from .sh to .ts. |
| memory/feedback_cross_platform_parity_hygiene_deferred_enforcement.md | Updates parity audit reference from .sh to .ts. |
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.
PR #1371 removed 13 .sh files; 6 memory files referenced 5 of them. Mechanical sed-replace .sh→.ts in those memos restores cross-reference edges. Decision-graph edge preservation per B-0141 framing.