feat(framework): make implementation commits obey project policy
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 55/100
- Issue type
- Feature
- Clarity
- Mostly clear
- Activity status
- Quiet
- Tech stack
- git, typescript
- Domain
- developer-experience, tooling
Research direction
Trace the VCS-memory template and implementation flow to find where commit policy is generated, resolved, and applied. Inspect the existing verification, branch, commit, and hook-handling paths first; done means absent or never prevents commits, enabled policies create atomic verified commits only on non-default branches, and hook rejection stops clearly without pushing or including unrelated changes.
Written by the indexing model from the issue text.
Description
Need
An AI implementing an approved plan may create small, reviewable local commits while it works. This behavior must be governed by the project policy, not hard-coded by the implementation flow.
Current gap
The VCS-memory template exposes commit-cycle choices, while the implementation flow always commits each verified phase. As a result, the documented never choice is not enforceable and the policy does not state the safety boundary.
Scope
- Project memory records one explicit automatic-commit policy, including when commits may occur and when they may not.
- The implementation flow resolves and obeys that policy. If it is absent or set to
never, it makes no automatic commit. - When enabled, an automatic commit contains one verified unit of work and follows the project VCS convention.
- Automatic commits run only on a non-default working branch. They never push, force-push, amend history, or include unrelated pre-existing changes.
- A rejected hook stops the flow and reports the reason; it does not retry blindly.
Out of scope
- Creating a new skill, hook, or background agent.
- Automatic pull requests, remote pushes, rebases, or branch deletion.
Acceptance criteria
- Generated VCS memory expresses the project policy in actionable terms, including the branch and push boundaries.
-
neverand an absent policy produce no automatic commit during implementation. - An enabled policy produces only atomic commits after the configured verification point.
- The implementation flow cannot automatically commit on the default branch or push a commit.
- A hook rejection leaves no new commit and gives a clear stop reason.
- Dominant language
- TypeScript
- Stars
- 481
- Forks
- 45
- Avg merge
- 14h 27m
- Merged PRs (30d)
- 111
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from ai-driven-dev/framework
-
refactor(aidd-orchestrator): the check zone says when to stop, and reviews its axes in one round Open
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
ai-driven-dev/framework#887 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
ai-driven-dev/framework#873 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
ai-driven-dev/framework#872 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
ai-driven-dev/framework#625 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
ai-driven-dev/framework#467 · 1 comment ·
All issues in ai-driven-dev/framework
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
safetrustcr/dApp-SafeTrust#426 ·
-
area:workflow bug ready-for-agent
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
fil-donadoni/tolaria#4409 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
Fission-AI/OpenSpec#1960 ·
-
Add dependabot Open
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
corsairdev/corsair#1764 ·