Skip to content

Memory-wiki cleanups (apply docstring, test grep, tradeoff docs) #137

Description

@nmrtn

Low-priority follow-ups from the #133 review:

  • --project usage docstring in bin/nanopm-ingest-agent shows --project after the subcommand (incl. the new apply line); argparse rejects that. Live code uses it correctly (project-first). Fix the docstring lines.
  • test/wiki-canonical.sh grep matched a local *.pyc; exclude *.pyc/binary so a dev who's run the bins doesn't hit a false failure (the new .gitignore stops the commit, not the local cache).
  • Document the accepted tradeoffs: apply is last-writer-wins (no read-modify-write); and a one-line note to drain/discard any existing wiki/_review/ before upgrading (the drainer was removed, so those are now orphaned).
  • CLAUDE.md still documents the retired nanopm_reasoning_pathReasoningFiles coupling as live — clean up.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions