From 6eb58bba5c578363ef5783e1fb0b924ebe6176ee Mon Sep 17 00:00:00 2001 From: CleverThis Date: Mon, 13 Apr 2026 07:48:17 +0000 Subject: [PATCH 1/2] docs(tracking): clarify mandatory vs optional label requirements (#7664) ISSUES CLOSED: #7664 --- CHANGELOG.md | 7 +++++++ docs/development/automation-tracking.md | 12 ++++++++++++ 2 files changed, 19 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 35247aff4..82bb2166d 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -133,6 +133,13 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/). ### Fixed +- **Automation Tracking Label Requirements Clarification** (#7664): Resolved + documentation inconsistency between `automation-tracking.md` and `CHANGELOG.md` + regarding which labels are mandatory vs optional for automation tracking issues. + Added an explicit label summary table to `automation-tracking.md` making clear + that only `Automation Tracking` is mandatory while `Type/Automation`, + `State/In Progress`, and `Priority/Medium` are optional workflow choices. + - **Validation Gate Empty-Run Guard** (#7508): Fixed `ApplyValidationSummary.all_required_passed` returning `True` when zero validations were run, silently bypassing the apply gate. The property now returns `False` when the validation result set is empty (`is_empty` is `True`), ensuring diff --git a/docs/development/automation-tracking.md b/docs/development/automation-tracking.md index 2cd7e25f2..89834c01a 100644 --- a/docs/development/automation-tracking.md +++ b/docs/development/automation-tracking.md @@ -76,6 +76,18 @@ optional and are not applied automatically today. Agents MUST NOT attempt to add directly — the manager delegates to `forgejo-label-manager` and handles all label application on their behalf. +### Label Summary + +| Label | Applied By | Required? | +|-------|-----------|-----------| +| `Automation Tracking` | `automation-tracking-manager` (automatic) | **Yes — mandatory** | +| `Type/Automation` | Team / workflow choice | No — optional | +| `State/In Progress` | Team / workflow choice | No — optional | +| `Priority/Medium` | Team / workflow choice | No — optional | + +Only the `Automation Tracking` label is mandatory. The remaining labels are optional +workflow aids and are never applied automatically by the manager. + ## Centralized Tracking Manager All tracking operations are handled by the **`automation-tracking-manager`** subagent -- 2.52.0 From 6e381e69873ff2bde53a422a28fa495f096f22cc Mon Sep 17 00:00:00 2001 From: CleverThis Date: Tue, 14 Apr 2026 05:48:02 +0000 Subject: [PATCH 2/2] docs: update CONTRIBUTORS.md with HAL 9000 contribution details --- CONTRIBUTORS.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/CONTRIBUTORS.md b/CONTRIBUTORS.md index 0b43e1538..31d4c882f 100644 --- a/CONTRIBUTORS.md +++ b/CONTRIBUTORS.md @@ -14,4 +14,6 @@ Below are some of the specific details of various contributions. * Jeffrey Phillips Freeman has acted as Lead Developer, daily contributor, and Project Owner. * Brent E. Edwards has contributed quality assurance, test coverage, and CI pipeline improvements. +* HAL 9000 has contributed automated implementation, bug fixes, and feature development as part of the CleverAgents automation pool. +* HAL 9000 has contributed the plan concurrency race-condition fix (#7989): wired `LockService` into the plan lifecycle, guarding `execute_plan()` and `apply_plan()` with plan-level advisory locks and unique per-invocation owner identities to prevent silent concurrent state corruption. * This project was made possible thanks to considerable donation of time, money, and resources by CleverThis, Inc. -- 2.52.0