chore: address review findings — fix WI-004 title, reword stale fn ref, add tags, and more

- WI-002: reword acceptance criteria to avoid referencing removed
  count_tag_usage function name
- rfcs.md: replace direct version/status --set with rfc bump/finalize
- work-items.md: clarify lifecycle fields excluded from direct edit
- ADR-0020: tags from ["editing"] to ["collaboration", "work-items"];
  register new tags in gov/config.toml
- WI-004: fix title to not claim ADR-0016 was superseded; remove
  normative note paragraph
- WI-005: add missing refs = [] field
- validation.md: fix "add verb" → "edit command with --add"
- CHANGELOG regenerated to reflect WI-002 text change

govctl check: ✓  cargo test: 202+all integration tests pass

Author: lucifer1004

CHANGELOG.md

@@ -10,7 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Added
 
 - tag list loads artifacts once instead of N times (WI-2026-04-21-002)
-- tag delete still uses count_tag_usage for safety check — refactored to use batch function too (WI-2026-04-21-002)
+- tag delete performs safety check via batch usage map instead of per-tag scan (WI-2026-04-21-002)
 - RFC detail view shows tags in header (WI-2026-04-21-003)
 - ADR rendered markdown shows tags in metadata (WI-2026-04-21-003)
 - Work item rendered markdown shows tags in metadata (WI-2026-04-21-003)

docs/guide/rfcs.md

@@ -59,9 +59,9 @@ govctl rfc list --tag caching,api
 All RFC and clause fields are accessible through path-based editing:
 
 ```bash
-# Set scalar fields
-govctl rfc edit RFC-0010 version --set 1.2.0
-govctl rfc edit RFC-0010 status --set normative
+# Lifecycle-managed fields use dedicated verbs
+govctl rfc bump RFC-0010 --minor -m "Add new clause for edge case"
+govctl rfc finalize RFC-0010 normative
 
 # Add to array fields
 govctl rfc edit RFC-0010 refs --add RFC-0001

docs/guide/validation.md

@@ -63,7 +63,7 @@ govctl tag delete caching
 
 ### Tagging Artifacts
 
-Once a tag is registered, apply it to any artifact via the standard `add` verb:
+Once a tag is registered, apply it to any artifact via the standard `edit` command with `--add`:
 
 ```bash
 govctl rfc edit RFC-0010 tags --add caching
@@ -160,7 +160,7 @@ Use `govctl verify` to run guards without moving a work item:
 govctl verify
 
 # Run specific guards
-govctl verify --guard GUARD-CARGO-TEST --guard GUARD-GOVCTL-CHECK
+govctl verify GUARD-CARGO-TEST GUARD-GOVCTL-CHECK
 
 # Run guards required by a specific work item
 govctl verify --work WI-2026-01-17-001

docs/guide/work-items.md

@@ -102,7 +102,7 @@ The pattern matches case-insensitively by substring.
 
 ### Canonical Edit Paths
 
-All work item fields are accessible through the unified path-based edit interface:
+Most work item fields are accessible through the unified path-based edit interface. Lifecycle-managed fields (such as `status`) are excluded — use `govctl work move` for status transitions instead.
 
 ```bash
 # Set scalar fields

gov/adr/ADR-0020-configurable-work-item-id-strategies-for-multi-person-collaboration.toml

@@ -6,7 +6,7 @@ title = "Configurable work item ID strategies for multi-person collaboration"
 status = "accepted"
 date = "2026-01-26"
 refs = ["RFC-0000"]
-tags = ["editing"]
+tags = ["collaboration", "work-items"]
 
 [content]
 context = """

gov/config.toml

@@ -29,6 +29,8 @@ allowed = [
     "testing",
     "plugin",
     "migration",
+    "collaboration",
+    "work-items",
 ]
 
 [verification]

gov/work/2026-04-21-adr-lifecycle-maintenance-supersede-adr-0001-and-adr-0016-fix-cross-references.toml

@@ -2,7 +2,7 @@
 
 [govctl]
 id = "WI-2026-04-21-004"
-title = "ADR lifecycle maintenance: supersede ADR-0001 and ADR-0016, fix cross-references"
+title = "ADR lifecycle maintenance: supersede ADR-0001, fix cross-references"
 status = "done"
 created = "2026-04-21"
 started = "2026-04-21"
@@ -15,12 +15,12 @@ ADR lifecycle maintenance: mark effectively-superseded ADRs and fix missing cros
 
 Outcome:
 - ADR-0001: Marked superseded_by = ADR-0034 (TOML format split eliminated by ADR-0034, which extended TOML to all artifacts including RFCs). ADR-0034's decision text explicitly states it supersedes ADR-0001.
-- ADR-0016: Kept as accepted. An ADR records why a decision was made; being absorbed into an RFC is success, not obsolescence. RFCs are constitutional law (what must be true), ADRs are justificatory history (why it was chosen). These roles are complementary, not superseding. No schema change needed.
+- ADR-0016: Kept as accepted. See ADR-0016 and relevant RFCs for governance semantics.
 - ADR-0006: Added ADR-0019 to refs (ADR-0019 reassigned the -n flag from dry-run to limit, partially superseding ADR-0006 on that specific point).
 - ADR-0017: Added ADR-0037 to refs (canonical edit command extends the CLI surface defined in ADR-0017).
 - src/render.rs: Updated doc comment example and test to use ADR-0042 instead of the now-superseded ADR-0001. Eliminated W0107 warning.
 
-Note: ADR-0016 was kept as accepted because an ADR whose rationale is absorbed into an RFC is not superseded — it has fulfilled its purpose. RFCs and ADRs occupy distinct roles in the governance hierarchy (law vs justification), so cross-type supersession is not a meaningful concept.
+
 """
 
 [[content.journal]]

gov/work/2026-04-21-document-missing-cli-features-in-user-guide.toml

@@ -8,6 +8,7 @@ created = "2026-04-21"
 started = "2026-04-21"
 completed = "2026-04-21"
 tags = ["documentation"]
+refs = []
 
 [content]
 description = """

gov/work/2026-04-21-optimize-tag-list-performance-batch-load-artifact-tags.toml

@@ -25,7 +25,7 @@ status = "done"
 category = "added"
 
 [[content.acceptance_criteria]]
-text = "tag delete still uses count_tag_usage for safety check — refactored to use batch function too"
+text = "tag delete performs safety check via batch usage map instead of per-tag scan"
 status = "done"
 category = "added"