ksamaschke/implementation-skills-claude
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

feat(implement-issues): add active-run heartbeat #4

Merged
ksamaschke merged 1 commit from feat/1-active-run-heartbeat into main 2026-07-06 04:01:47 +00:00
Conversation 0 Commits 1 Files changed 4 +130 -48
ksamaschke commented 2026-07-06 04:01:21 +00:00
Owner
Copy link

Closes #1

Acceptance criteria:

  • Heartbeat documented before first git command, with per-pass run_id, atomic lock/temp/rename/re-read acquisition, phase refreshes, and cleanup.
  • Fresh foreign heartbeat stands down with another implement-issues run is active; standing down.
  • 24 h stale handling uses ISO-8601 UTC and defines missing/unparseable handling.
  • Active/lock/temp files are repo-local and never staged; .gitignore and .git/info/exclude protections are documented and applied locally.
  • Tracker-only stale-own-claim scan happens before new claims.
  • Claim-before-git-work ordering is documented for GitHub/Forgejo, with a file-backend metadata-sync exception and push-rejection re-read behavior.

Verification:

  • git diff --check -- .gitignore plugins/implementation-skills/skills/implement-issues/SKILL.md plugins/implementation-skills/skills/implement-issues/references/project-config.md plugins/implementation-skills/skills/implement-issues/references/trackers.md
  • targeted rg gate for heartbeat, run_id, lock/temp, stale timestamps, claim ordering, and file-backend activity semantics
  • git check-ignore -v .claude/implement-issues.active .claude/implement-issues.active.lock/foo .claude/implement-issues.active.tmp.abc.123
  • independent requirements check: PASS
  • independent hostile review: no blocking findings; final nits fixed
Closes #1 Acceptance criteria: - [x] Heartbeat documented before first git command, with per-pass `run_id`, atomic lock/temp/rename/re-read acquisition, phase refreshes, and cleanup. - [x] Fresh foreign heartbeat stands down with `another implement-issues run is active; standing down`. - [x] 24 h stale handling uses ISO-8601 UTC and defines missing/unparseable handling. - [x] Active/lock/temp files are repo-local and never staged; `.gitignore` and `.git/info/exclude` protections are documented and applied locally. - [x] Tracker-only stale-own-claim scan happens before new claims. - [x] Claim-before-git-work ordering is documented for GitHub/Forgejo, with a file-backend metadata-sync exception and push-rejection re-read behavior. Verification: - `git diff --check -- .gitignore plugins/implementation-skills/skills/implement-issues/SKILL.md plugins/implementation-skills/skills/implement-issues/references/project-config.md plugins/implementation-skills/skills/implement-issues/references/trackers.md` - targeted `rg` gate for heartbeat, run_id, lock/temp, stale timestamps, claim ordering, and file-backend activity semantics - `git check-ignore -v .claude/implement-issues.active .claude/implement-issues.active.lock/foo .claude/implement-issues.active.tmp.abc.123` - independent requirements check: PASS - independent hostile review: no blocking findings; final nits fixed
ksamaschke deleted branch feat/1-active-run-heartbeat 2026-07-06 04:01:47 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
ksamaschke/implementation-skills-claude!4
No description provided.