Skip to content

chore: retire package.k9, a manifest nothing consumed (DEBT D4) - #61

Merged
hyperpolymath merged 1 commit into
mainfrom
chore/retire-package-k9
Oct 6, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
chore/retire-package-k9

Conversation

@hyperpolymath

Copy link
Copy Markdown
Contributor

Summary

This PR deletes package.k9, a 20-line "Must-Just-Nickel" manifest that nothing in the repo reads, and clears DEBT D4.

  • No code, script, workflow or Justfile reads the file. It was the only .k9 file in the tree.
  • Its just build target named a recipe that had already been deleted.
  • D4 offered three ways to clear it: wire it into CI, mark it as declaration-only, or delete it. This PR takes the smallest, deletion, at the owner's request.

References that went with it:

  • dust probe (version drift): dropped package.k9 from its grep. With the file gone, grep would exit 2, and the probe's leading ! would turn that into a pass that checks nothing.
  • must.ncl, chora deed: removed the package.k9 sentence and the (supersedes …) entry, and rewrote the neurosym comment.
  • AGENTS.md, wiki Architecture.adoc (table row), AUDIT.adoc: removed the references or reworded them in the past tense.
  • DEBT.adoc: D4 is marked CLEARED 2026-10-06, with its evidence. The grep shown as evidence for L1 no longer names a file that has gone.
  • CHANGELOG.md: added an Unreleased → Removed entry.

Closes # — no issue; this clears DEBT D4.

Type of change

  • 🐛 Bug fix: no behaviour change.
  • ✨ New feature: none.
  • 💥 Breaking change: none. Nothing consumed the file.
  • 🕳️ Soundness fix: not a checker fix, though it stops one probe from passing vacuously (see above).
  • 📖 Documentation
  • 🧹 Refactor / tech debt: removes a dead file (D4).
  • ⚡ Performance: none.
  • 🔧 Build / CI / tooling: the probe is documentary, so no CI wiring changes.

📌 New pins

  • Head SHA: dd297ea
  • New or changed pins: none.

How has this been verified?

  • bun install --frozen-lockfile && bun test: 86 pass, 0 fail.
  • just probes: 36 pass, 0 fail.
  • Dust probe mutant: I ran the new run line on the clean tree (rc 0), then with V="0.1.0" planted in progblocks-launcher.sh (rc 1, caught). I reverted the plant and checked it was gone with git diff --quiet.
  • git grep -n package.k9: the remaining hits are all history-tense lines in the CHANGELOG, DEBT, AUDIT, the deed comment, the dust description and RELEASE-CRITERIA T2.
  • Independence: git grep -niE 'berry|a2ml' src/ finds nothing.

Checklist

  • My commits are signed: a good ED25519 signature on dd297ea.
  • I ran the project's own checks locally (tests, probes), and they pass.
  • New files carry the correct SPDX identifier: there are no new files; this PR only deletes and edits.
  • Docs are updated, and no claim overstates the code.
  • No soundness hole introduced: the vacuous-pass path in the probe is closed, and the mutant check above shows it.

Notes for reviewers

  • The published GitHub wiki mirrors docs/wiki/, so the Architecture row drops out on the next tools/publish-wiki.mjs run. It needs no hand edit.

🤖 Generated with Claude Code

https://claude.ai/code/session_019tmsGh5n4GwJZYq9Z5mtxH

No code, script, workflow or Justfile reads package.k9, and its `just build`
target named a recipe that was deleted. Delete it, of D4's three options the
smallest, and remove it from the dust version-drift probe (whose grep would
otherwise exit 2 on the missing file and pass vacuously), the must.ncl and
chora deed text, AGENTS.md, AUDIT, DEBT, the wiki's Architecture table and the
CHANGELOG.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019tmsGh5n4GwJZYq9Z5mtxH
@coderabbitai

coderabbitai Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: a8567b03-6b82-443b-9458-16daf24c26e7
📥 Commits

Reviewing files that changed from the base of the PR and between aad4046 and dd297ea.

📒 Files selected for processing (9)
  • .machine_readable/contractiles/dust/dust.ncl
  • .machine_readable/contractiles/must/must.ncl
  • .machine_readable/descriptiles/progblocks_chora.deed
  • AGENTS.md
  • AUDIT.adoc
  • CHANGELOG.md
  • DEBT.adoc
  • docs/wiki/developers/Architecture.adoc
  • package.k9
 __________________________________________
< Performing energy-intensive code review. >
 ------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@hyperpolymath
hyperpolymath merged commit 8bce4aa into main Oct 6, 2026
13 of 14 checks passed
@hyperpolymath
hyperpolymath deleted the chore/retire-package-k9 branch October 6, 2026 22:49
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