Skip to content

Track memory-bank as T0 resume SSOT - #145

Merged
cryptoxdog merged 2 commits into
mainfrom
chore/track-memory-bank
Jul 25, 2026
Merged

Track memory-bank as T0 resume SSOT#145
cryptoxdog merged 2 commits into
mainfrom
chore/track-memory-bank

Conversation

@cryptoxdog

Copy link
Copy Markdown
Collaborator

Summary

  • Add memory-bank/ (activeContext, tasks, progress, tech-debt) to the repo so session resume state is not local-only after a fresh clone
  • Document in .gitignore that memory-bank/ must remain tracked (do not re-add to repo or ~/.gitignore_global)

Context

memory-bank/ was previously ignored via ~/.gitignore_global. That ignore was removed; this PR finishes the un-ignore by actually tracking the files.

Companion to #144 (agent docs wiring). Scoped separately so resume-SSOT tracking can merge independently of the docs/tooling refactor.

Test plan

  • git check-ignore -v memory-bank/activeContext.md exits 1 (not ignored)
  • Fresh clone shows memory-bank/ present
  • Confirm ~/.gitignore_global has no memory-bank entry

Made with Cursor

After removing memory-bank from the global gitignore, add the directory to
the repo and document that it must stay tracked.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

github-actions Bot commented Jul 24, 2026

Copy link
Copy Markdown

⚠️ Large PR Warning
Reviewable lines changed: 410
Warning threshold: 300 lines
Consider splitting for easier review

📋 Best Practices for Large Changes

  1. Refactoring + Features: Separate into 2 PRs
  2. Multiple Features: One PR per feature
  3. Database + Code: Separate migration from logic
  4. Generated Code: Exclude it from reviewable-size accounting

This PR passes the blocking limit but is larger than recommended.

The four files were last written on this branch before the L9_META pipeline and
contract-registry work landed, so the resume state pointed at PRs that have since
been opened and at next actions that are done.

activeContext.md is rewritten rather than appended: a session-end hook had
overwritten it in the main working tree with a bare template ("No summary
provided" plus a generic file list), which would have destroyed the PR #144
narrative this branch already carried. The rewrite keeps that context, folds in
#145-#148, and records the one non-obvious hazard — PR #147 already contains the
full-repo header stamp, so a second "meta-only" PR would duplicate it.

progress.md, tasks.md and tech-debt.md are additive; the three tech-debt entries
that appear removed are replaced by updated versions in the same commit.

Co-authored-by: Cursor <cursoragent@cursor.com>
@sonarqubecloud

Copy link
Copy Markdown

@cryptoxdog
cryptoxdog merged commit a73d122 into main Jul 25, 2026
38 of 44 checks passed
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