Skip to content

fix: ignore downstream start-issue run ledgers - #116

Merged
dapi merged 1 commit into
mainfrom
agent/ignore-start-issue-run-ledgers
Aug 12, 2026
Merged

fix: ignore downstream start-issue run ledgers#116
dapi merged 1 commit into
mainfrom
agent/ignore-start-issue-run-ledgers

Conversation

@dapi

@dapi dapi commented Aug 12, 2026

Copy link
Copy Markdown
Owner

What changed

  • add a nested template/.start-issue/.gitignore containing runs/
  • keep downstream root .gitignore files untouched
  • let the existing managed-payload install/update contract create the nested rule on both first adoption and later updates

Why

.start-issue/prompt.md already requires Run Ledger state under .start-issue/runs/ to be ignored by Git. The ignore rule existed only in this upstream repository's root .gitignore, which is outside template/ and therefore is not installed downstream. Putting the rule beside the generated state makes it an independent managed payload path, so it does not collide with a project's user-owned root .gitignore.

Workflow: Bug Fix

  • Expected behavior: template/.start-issue/prompt.md states that .start-issue/runs/ is Git-ignored before the ledger is written.
  • Actual behavior: a downstream repository with its own root .gitignore receives .start-issue/prompt.md but does not ignore the run ledger.
  • Root cause: the only effective rule was in the upstream repository root, outside the installed template/ payload.
  • Validation profile: standard; this changes installed template behavior without changing the CLI ownership contract or publishing a release.

Downstream impact

  • Fresh memory-bank-cli init: creates the nested ignore file as managed, while preserving an existing root .gitignore as user-owned.
  • Existing memory-bank-cli update: creates the new nested ignore file as a non-conflicting managed path, even when the root .gitignore remains user-owned.
  • Repeated updates preserve the managed file through the existing ownership lock semantics.

No memory-bank-cli change or coordinated release is required.

Verification

  • ruby tools/validate-priming-manifests-test.rb
  • ruby tools/validate-priming-manifests.rb template/memory-bank
  • memory-bank-cli lint --scope-root template/memory-bank --entrypoint template/memory-bank/README.md
  • memory-bank-cli doctor --profile template
  • git diff --check origin/main...HEAD
  • end-to-end install with memory-bank-cli 1.8.1 into a Git repository with a custom root .gitignore: nested rule created as managed; root file preserved; git check-ignore identifies .start-issue/.gitignore
  • end-to-end update from pre-fix origin/main to this branch under the same conditions: nested rule created as managed; root file remains user-owned and unchanged; git check-ignore identifies .start-issue/.gitignore

@dapi
dapi marked this pull request as ready for review August 12, 2026 21:26
@dapi
dapi merged commit daa7cb6 into main Aug 12, 2026
1 check passed
@dapi
dapi deleted the agent/ignore-start-issue-run-ledgers branch August 12, 2026 21:26
dapi added a commit that referenced this pull request Aug 13, 2026
…fecycle

* origin/main:
  fix: ignore start-issue run ledgers downstream (#116)
  docs: introduce Memory Bank in English
  docs: add MECE governance principle and audit (#115)
  docs: explain BDD and use case relationships
dapi added a commit that referenced this pull request Aug 14, 2026
…-sistematizirovat-trebovaniya-i-tras

* origin/main: (21 commits)
  Remove reduntant files from memory-bank
  docs: close FT-117 lifecycle
  docs: refresh FT-117 merged evidence
  docs: complete FT-117 recovery review
  docs: complete autonomous decision protocol (#117)
  docs: clarify FT-113 review evidence
  docs: close FT-113 lifecycle
  docs: record FT-113 execution state
  docs: preserve BDD methodology source
  docs: activate FT-113 implementation plan
  docs: add FT-113 delivery package
  docs: make decision protocol self-contained (#117)
  docs: address autonomy review findings (#117)
  docs: make agent decisions autonomous by default (#117)
  fix: ignore start-issue run ledgers downstream (#116)
  docs: introduce Memory Bank in English
  docs: add MECE governance principle and audit (#115)
  docs: integrate BDD lifecycle practice
  docs: explain BDD and use case relationships
  docs: add 4+1 viewpoint coverage (#112)
  ...

# Conflicts:
#	template/memory-bank/flows/feature-artifact-catalog.md
#	template/memory-bank/flows/feature.md
#	template/memory-bank/flows/priming/feature.yaml
#	template/memory-bank/flows/templates/feature/brief.md
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