Skip to content

feat: selective auto-tag at idd-issue baseline + idd-verify PASS milestones#250

Merged
kiki830621 merged 2 commits into
mainfrom
idd/85-auto-tag
Jul 10, 2026
Merged

feat: selective auto-tag at idd-issue baseline + idd-verify PASS milestones#250
kiki830621 merged 2 commits into
mainfrom
idd/85-auto-tag

Conversation

@kiki830621

Copy link
Copy Markdown
Member

Refs #85

Summary

Selective git auto-tag at two IDD milestones: idd-issue tags idd-{N}-baseline (main HEAD = rollback anchor), idd-verify tags idd-{N}-verified on Aggregate PASS (review snapshot). Only these two points are tagged (no diagnose/plan/implement tags) to keep the namespace clean. Config auto_tag (default-ON, opt-out via enabled:false); idempotent + graceful-skip on push failure (never aborts).

Checklist

  • Diagnose (Plan tier)
  • Plan (approved: default-ON + PR path)
  • Implement (1 commit, 32008b7; auto-tag drift-guard 27/27, full suite green)
  • Verify (run /idd-verify #85 --pr <this PR>)
  • Verify-gated: post-verify PASS = ready to merge → after merge, run /idd-close to finalize (manual gate + closing summary; no auto-close trailer)

Generated by /idd-implement on PR path. Do NOT add a GitHub close trailer — IDD discipline requires manual /idd-close after merge.

…85)

idd-issue tags idd-{N}-baseline at main HEAD (rollback anchor); idd-verify
tags idd-{N}-verified at the Aggregate-PASS snapshot (review-ready). Only
these two milestones (no diagnose/plan/implement tags) so the tag namespace
stays clean. Config auto_tag (default-on, opt-out via enabled:false);
idempotent (existing tag skipped) + graceful-skip on push failure (never
aborts the workflow). New auto-tag drift-guard, 27 assertions.

Refs #85
…HEAD (#85)

Self-review of PR #250 found two defects in the idd-verify Step 4.5 prose:
- $CONFIG_PATH was undefined (idd-verify has no find_idd_config, unlike
  idd-issue) — now resolves the walked-up config inline (new path first,
  legacy fallback).
- $INPUT_SOURCE / $PR_HEAD_SHA were undefined AND unnecessary: tag_verified
  runs before restore_working_tree, so HEAD is already the PR head in PR mode
  (gh pr checkout done in Step 0.5) and the verified tree in local mode.
  Snapshot simplifies to HEAD.

Drift-guard stays 27/27.

Refs #85
@kiki830621 kiki830621 merged commit df99960 into main Jul 10, 2026
1 check passed
@kiki830621 kiki830621 deleted the idd/85-auto-tag branch July 10, 2026 00:21
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