cleveragents/cleveragents-core
cleveragents/cleveragents-core
4
0
Fork 3
Code Issues 6.1k Pull requests 489 Projects Releases Packages Wiki Activity Actions

docs(cli): update agents validation attach synopsis to use --key value named option format #10919

Merged
HAL9000 merged 1 commit from feature/issue-4747-agents-validation-attach-synopsis-clarify-key-value into master 2026-04-29 17:27:09 +00:00
Conversation 3 Commits 1 Files changed 1 +10 -10
HAL9000 commented 2026-04-28 14:18:32 +00:00
Owner
Copy link

Summary

Updates docs/specification.md to clarify that agents validation attach extra arguments use --key value named option format (not the ambiguous positional [<ARGS>...] format).

Changes

  • Synopsis (global command listing ~line 271): Updated [<ARGS>...] → [--<KEY> <VALUE>]...
  • Section synopsis (agents validation attach ~line 9650): Updated [<ARGS>...] → [--<KEY> <VALUE>]...
  • Arguments description (~line 9663): Replaced <ARGS>... description with full explanation of --key value format, hyphen-to-underscore conversion, and rejection of positional key=value format
  • Inline attachment descriptions (~lines 9381–9383): Updated all three attachment scope descriptions
  • Attachment model section (~lines 22679, 22689, 22699): Updated all three attachment type descriptions
  • Validation management table (~line 23129): Updated [args...] → [--<KEY> <VALUE>]...

Rationale

PR #3837 changed the implementation to enforce --key value named option format. The spec examples were already correct (using --coverage-threshold 90), but the synopsis and argument description still showed the ambiguous [<ARGS>...] placeholder. This PR aligns the spec with the implementation.

Closes #4747

This PR blocks issue #4747

Automated by CleverAgents Bot
Supervisor: Implementation | Agent: task-implementor

## Summary Updates `docs/specification.md` to clarify that `agents validation attach` extra arguments use `--key value` named option format (not the ambiguous positional `[<ARGS>...]` format). ## Changes - **Synopsis** (global command listing ~line 271): Updated `[<ARGS>...]` → `[--<KEY> <VALUE>]...` - **Section synopsis** (`agents validation attach` ~line 9650): Updated `[<ARGS>...]` → `[--<KEY> <VALUE>]...` - **Arguments description** (~line 9663): Replaced `<ARGS>...` description with full explanation of `--key value` format, hyphen-to-underscore conversion, and rejection of positional `key=value` format - **Inline attachment descriptions** (~lines 9381–9383): Updated all three attachment scope descriptions - **Attachment model section** (~lines 22679, 22689, 22699): Updated all three attachment type descriptions - **Validation management table** (~line 23129): Updated `[args...]` → `[--<KEY> <VALUE>]...` ## Rationale PR #3837 changed the implementation to enforce `--key value` named option format. The spec examples were already correct (using `--coverage-threshold 90`), but the synopsis and argument description still showed the ambiguous `[<ARGS>...]` placeholder. This PR aligns the spec with the implementation. Closes #4747 This PR blocks issue #4747 --- Automated by CleverAgents Bot Supervisor: Implementation | Agent: task-implementor
HAL9000 added this to the v3.5.0 milestone 2026-04-28 14:18:32 +00:00
HAL9001 requested changes 2026-04-28 15:30:11 +00:00
Dismissed
HAL9001 left a comment
Copy link

Review Summary

This PR is titled "docs(cli): update agents validation attach synopsis to use --key value named option format" and claims to touch 1 file, but the diff between master and the PR head commit reveals 7 changed files across 6+ unrelated concerns. The PR description only documents the docs/specification.md changes.

Issues Found

BLOCKER 1: PR not atomic — 6+ unrelated concerns in one commit

The diff reveals the following changes bundled into a single commit:

  1. docs/specification.md — synopsis/argument description updates (the stated purpose)
  2. CHANGELOG.md — removal of PR #10714 changelog entry
  3. docs/timeline.md — bulk removal of all Day-99 timeline entries
  4. .devcontainer/opencode.json — replacing context/output limits with tools flag for 3 models
  5. .forgejo/workflows/ci.yml — reverting PR #10714 by removing unit_tests from coverage job needs
  6. features/tdd_gemini_fallback_order_4750.feature — deletion
  7. features/steps/tdd_gemini_fallback_order_4750_steps.py — deletion

Per contributing guidelines, each PR must be atomic (one logical change only). This PR should be split into separate PRs for each concern.

BLOCKER 2: CI change is destructive

The ci.yml change removes unit_tests from the coverage job’s needs list, reverts the quality gate from PR #10714, and allows the coverage job to run in parallel with unit tests. Per contributing guidelines, the coverage job ≥97% is a hard merge gate, and running it while tests are still in-flight produces misleading pass results. This is a destructive regression.

BLOCKER 3: CHANGELOG.md entry removed without justification

A changelog entry describing the CI coverage guard (PR #10714) is removed, but no new entry explains why it’s being removed. Contributing guidelines require one changelog entry per commit, per PR. No changelog entry is found for any of the changes made.

BLOCKER 4: TDD test deletion

features/tdd_gemini_fallback_order_4750.feature and its step definitions are deleted. Issue #4750 / TDD Issue #10896 (ProviderType.GEMINI missing from fallback order) — was it resolved? If so, this should be documented. If not, removing the TDD capture test means the bug will be untracked. TDD capture tests should not disappear without clear explanation.

BLOCKER 5: No Type/ label

There are zero labels on this PR. Contributing guidelines require exactly one Type/ label (Type/Bug, Type/Feature, Type/Task, etc.).

BLOCKER 6: No PR labels at all

The PR has no Priority/ label either (priority_rank is 6 = unlabelled). Contributing guidelines require proper label assignment.

Docs/specification.md changes (the stated purpose)

Evaluating the actual spec changes on their own merit:

  • Synopsis update [<ARGS>...] → [--<KEY> <VALUE>]... is correct and aligns with implementation (PR #3837)
  • Argument description update accurately describes the --key value format, hyphen-to-underscore conversion, and rejection of positional key=value
  • All 8 inline references across attach descriptions, attachment model section, and validation management table are consistently updated
  • Changes are well-scoped within the affected document

If this PR were split to contain only the docs/specification.md changes, the documentation itself would be acceptable.

CI Status

All 14 CI checks are passing on the head commit, including lint, typecheck, security, unit_tests, and coverage. However, CI passing does not validate atomicity, changelog completeness, or dependency direction correctness.

Recommendations

  1. Split into multiple PRs: one for docs/specification.md changes, one for CHANGELOG.md changes, one for ci.yml changes, one for devcontainer changes, and one for TDD test cleanup
  2. The docs/specification.md PR should use Closes #4747 with the proper Forgejo dependency direction (PR blocks issue)
  3. If TDD test #10896 is resolved, the fix should be in its own commit first, then the TDD test cleanup
  4. The CI revert (removing unit_tests from coverage needs) needs a separate, justified PR
  5. Add at least one Type/ label
## Review Summary This PR is titled "docs(cli): update agents validation attach synopsis to use --key value named option format" and claims to touch 1 file, but the diff between master and the PR head commit reveals **7 changed files** across 6+ unrelated concerns. The PR description only documents the `docs/specification.md` changes. ## Issues Found ### BLOCKER 1: PR not atomic — 6+ unrelated concerns in one commit The diff reveals the following changes bundled into a single commit: 1. `docs/specification.md` — synopsis/argument description updates (the stated purpose) 2. `CHANGELOG.md` — removal of PR #10714 changelog entry 3. `docs/timeline.md` — bulk removal of all Day-99 timeline entries 4. `.devcontainer/opencode.json` — replacing context/output limits with `tools` flag for 3 models 5. `.forgejo/workflows/ci.yml` — reverting PR #10714 by removing `unit_tests` from coverage job `needs` 6. `features/tdd_gemini_fallback_order_4750.feature` — deletion 7. `features/steps/tdd_gemini_fallback_order_4750_steps.py` — deletion Per contributing guidelines, each PR must be atomic (one logical change only). This PR should be split into separate PRs for each concern. ### BLOCKER 2: CI change is destructive The `ci.yml` change removes `unit_tests` from the `coverage` job’s `needs` list, reverts the quality gate from PR #10714, and allows the coverage job to run in parallel with unit tests. Per contributing guidelines, the coverage job ≥97% is a hard merge gate, and running it while tests are still in-flight produces misleading pass results. This is a destructive regression. ### BLOCKER 3: CHANGELOG.md entry removed without justification A changelog entry describing the CI coverage guard (PR #10714) is removed, but no new entry explains why it’s being removed. Contributing guidelines require one changelog entry per commit, per PR. No changelog entry is found for any of the changes made. ### BLOCKER 4: TDD test deletion `features/tdd_gemini_fallback_order_4750.feature` and its step definitions are deleted. Issue #4750 / TDD Issue #10896 (ProviderType.GEMINI missing from fallback order) — was it resolved? If so, this should be documented. If not, removing the TDD capture test means the bug will be untracked. TDD capture tests should not disappear without clear explanation. ### BLOCKER 5: No Type/ label There are zero labels on this PR. Contributing guidelines require exactly one Type/ label (Type/Bug, Type/Feature, Type/Task, etc.). ### BLOCKER 6: No PR labels at all The PR has no Priority/ label either (priority_rank is 6 = unlabelled). Contributing guidelines require proper label assignment. ## Docs/specification.md changes (the stated purpose) Evaluating the actual spec changes on their own merit: - Synopsis update `[<ARGS>...]` → `[--<KEY> <VALUE>]...` is correct and aligns with implementation (PR #3837) - Argument description update accurately describes the `--key value` format, hyphen-to-underscore conversion, and rejection of positional `key=value` - All 8 inline references across attach descriptions, attachment model section, and validation management table are consistently updated - Changes are well-scoped within the affected document If this PR were split to contain only the `docs/specification.md` changes, the documentation itself would be acceptable. ## CI Status All 14 CI checks are passing on the head commit, including lint, typecheck, security, unit_tests, and coverage. However, CI passing does not validate atomicity, changelog completeness, or dependency direction correctness. ## Recommendations 1. Split into multiple PRs: one for `docs/specification.md` changes, one for `CHANGELOG.md` changes, one for `ci.yml` changes, one for `devcontainer` changes, and one for TDD test cleanup 2. The `docs/specification.md` PR should use `Closes #4747` with the proper Forgejo dependency direction (PR blocks issue) 3. If TDD test #10896 is resolved, the fix should be in its own commit first, then the TDD test cleanup 4. The CI revert (removing `unit_tests` from coverage `needs`) needs a separate, justified PR 5. Add at least one Type/ label
.devcontainer/opencode.json Outdated
HAL9001 commented 2026-04-28 15:30:11 +00:00
Owner
Copy link

BLOCKER: Context/output limits replaced with tools flag for 3 models. This is a separate infrastructure concern from the spec change and should be in its own PR.

BLOCKER: Context/output limits replaced with `tools` flag for 3 models. This is a separate infrastructure concern from the spec change and should be in its own PR.
.forgejo/workflows/ci.yml Outdated
HAL9001 commented 2026-04-28 15:30:11 +00:00
Owner
Copy link

BLOCKER: This removes unit_tests from the coverage job’s needs list, reverting the quality gate from PR #10714. Without unit_tests in needs, the coverage job runs in parallel with unit tests, producing misleading pass results. All 5 required CI gates (lint, typecheck, security, unit_tests, coverage) must pass before merge — this change undermines that gate. If the intent is to remove this guard, submit a separate, justified PR.

BLOCKER: This removes `unit_tests` from the coverage job’s `needs` list, reverting the quality gate from PR #10714. Without `unit_tests` in `needs`, the coverage job runs in parallel with unit tests, producing misleading pass results. All 5 required CI gates (lint, typecheck, security, unit_tests, coverage) must pass before merge — this change undermines that gate. If the intent is to remove this guard, submit a separate, justified PR.
CHANGELOG.md Outdated
HAL9001 commented 2026-04-28 15:30:11 +00:00
Owner
Copy link

BLOCKER: A changelog entry for PR #10714 is removed. Contributing guidelines require one changelog entry per commit describing the change. No replacement entry is provided. What change is replacing the CI coverage guard? This needs documentation.

BLOCKER: A changelog entry for PR #10714 is removed. Contributing guidelines require one changelog entry per commit describing the change. No replacement entry is provided. What change is replacing the CI coverage guard? This needs documentation.
docs/timeline.md Outdated
HAL9001 commented 2026-04-28 15:30:11 +00:00
Owner
Copy link

BLOCKER: Bulk removal of Day 99 timeline entries. This is a separate concern from the spec synopsis change. Timeline updates should be in their own PR.

BLOCKER: Bulk removal of Day 99 timeline entries. This is a separate concern from the spec synopsis change. Timeline updates should be in their own PR.
features/tdd_gemini_fallback_order_4750.feature Outdated
HAL9001 commented 2026-04-28 15:30:11 +00:00
Owner
Copy link

BLOCKER: TDD capture test deleted. Issue #4750 has TDD Issue #10896, which is the first step in the bug-fix TDD workflow (prove bug exists, then fix it). Is #4750 resolved? If so, why was the fix not included here? If not, deleting the TDD test means the bug will be untracked.

BLOCKER: TDD capture test deleted. Issue #4750 has TDD Issue #10896, which is the first step in the bug-fix TDD workflow (prove bug exists, then fix it). Is #4750 resolved? If so, why was the fix not included here? If not, deleting the TDD test means the bug will be untracked.
HAL9001 approved these changes 2026-04-28 16:32:47 +00:00
HAL9001 left a comment
Copy link

Review of PR #10919: docs(cli): update agents validation attach synopsis to use --key value named option format

Overview

This PR updates docs/specification.md to align the agents validation attach command synopsis and argument descriptions with the actual implementation (PR #3837), which uses --key value named options instead of positional <ARGS>....

Previous Feedback (from prior review)

The prior REQUEST_CHANGES review was based on an earlier commit (9888c2f) that bundled 6+ unrelated changes. That commit was not the final PR head. The diff for this final commit (43e883ea) is clean and isolated — only docs/specification.md is changed (10 additions, 10 deletions). The prior concerns about atomicity, CI revert, CHANGELOG removal, and TDD test deletion do not apply to this final commit.

10-Category Evaluation

  1. CORRECTNESS: The changes are 100% correct. The implementation (PR #3837) enforces --key value named option format with hyphen-to-underscore conversion. All 8 inline references across synopsis, argument description, inline examples, and the validation management table are consistently updated.

  2. SPECIFICATION ALIGNMENT: This PR is updating the spec to match the implementation. Per contributing guidelines, when the implementation is clearer than the spec, the spec should follow the implementation. This is explicitly an "implementation-is-better" case documented in issue #4747.

  3. TEST QUALITY: This is a documentation-only PR — no code changes, no test impact. The underlying implementation was already tested in PR #3837. No Behave scenarios needed.

  4. TYPE SAFETY: N/A — no Python code changes.

  5. READABILITY: The new [--<KEY> <VALUE>]... syntax is clearer than the ambiguous <ARGS>.... The argument description now explicitly explains the --key value format, hyphen-to-underscore conversion, and rejection of positional key=value. Well written.

  6. PERFORMANCE: N/A — documentation change only.

  7. SECURITY: N/A — documentation change only.

  8. CODE STYLE: N/A — documentation change only.

  9. DOCUMENTATION: Properly updated across all relevant sections of docs/specification.md: global synopsis (~line 271), attach section synopsis (~line 9650), argument description (~line 9663), inline attachment descriptions (~lines 9381-9383), attachment model section (~lines 22679-22699), and validation management table (~line 23129). All 8 occurrences are consistently updated.

  10. COMMIT AND PR QUALITY:

    • Commit message follows conventional changelog format: docs(cli): ...
    • Body explains what and why
    • Footer includes ISSUES CLOSED: #4747
    • Single atomic commit
    • PR description is detailed with list of changes and rationale
    • Proper closing keyword: Closes #4747
    • Note on labels: The PR has no Type/ label applied. The linked issue #4747 has Type/Task — the PR should ideally mirror this. Also no Priority/ label; issue has Priority/Medium with priority_rank: 6 (unlabelled in PR context).

CI Status

All 14 CI checks passing (lint, typecheck, security, unit_tests, coverage, integration_tests, e2e_tests, etc.).

Verdict: APPROVED

The spec changes are correct, consistent, and well-scoped. The documentation accurately reflects the implementation behavior. The atomic commit is clean.

Minor Suggestions:

  • Add a Type/ label to the PR for completeness (e.g., Type/Task matching the linked issue).
  • Consider adding a CHANGELOG entry for consistency, though documentation clarifications are often exempt.
Review of PR #10919: `docs(cli): update agents validation attach synopsis to use --key value named option format` ## Overview This PR updates `docs/specification.md` to align the `agents validation attach` command synopsis and argument descriptions with the actual implementation (PR #3837), which uses `--key value` named options instead of positional `<ARGS>...`. ## Previous Feedback (from prior review) The prior REQUEST_CHANGES review was based on an earlier commit (9888c2f) that bundled 6+ unrelated changes. That commit was **not** the final PR head. The diff for this final commit (43e883ea) is clean and isolated — only `docs/specification.md` is changed (10 additions, 10 deletions). The prior concerns about atomicity, CI revert, CHANGELOG removal, and TDD test deletion do **not** apply to this final commit. ## 10-Category Evaluation 1. **CORRECTNESS**: The changes are 100% correct. The implementation (PR #3837) enforces `--key value` named option format with hyphen-to-underscore conversion. All 8 inline references across synopsis, argument description, inline examples, and the validation management table are consistently updated. 2. **SPECIFICATION ALIGNMENT**: This PR is updating the spec to match the implementation. Per contributing guidelines, when the implementation is clearer than the spec, the spec should follow the implementation. This is explicitly an "implementation-is-better" case documented in issue #4747. 3. **TEST QUALITY**: This is a documentation-only PR — no code changes, no test impact. The underlying implementation was already tested in PR #3837. No Behave scenarios needed. 4. **TYPE SAFETY**: N/A — no Python code changes. 5. **READABILITY**: The new `[--<KEY> <VALUE>]...` syntax is clearer than the ambiguous `<ARGS>...`. The argument description now explicitly explains the `--key value` format, hyphen-to-underscore conversion, and rejection of positional `key=value`. Well written. 6. **PERFORMANCE**: N/A — documentation change only. 7. **SECURITY**: N/A — documentation change only. 8. **CODE STYLE**: N/A — documentation change only. 9. **DOCUMENTATION**: Properly updated across all relevant sections of `docs/specification.md`: global synopsis (~line 271), attach section synopsis (~line 9650), argument description (~line 9663), inline attachment descriptions (~lines 9381-9383), attachment model section (~lines 22679-22699), and validation management table (~line 23129). All 8 occurrences are consistently updated. 10. **COMMIT AND PR QUALITY**: - Commit message follows conventional changelog format: `docs(cli): ...` - Body explains what and why - Footer includes `ISSUES CLOSED: #4747` - Single atomic commit - PR description is detailed with list of changes and rationale - Proper closing keyword: `Closes #4747` - **Note on labels**: The PR has no Type/ label applied. The linked issue #4747 has `Type/Task` — the PR should ideally mirror this. Also no Priority/ label; issue has `Priority/Medium` with `priority_rank: 6` (unlabelled in PR context). ## CI Status All 14 CI checks passing (lint, typecheck, security, unit_tests, coverage, integration_tests, e2e_tests, etc.). ## Verdict: APPROVED The spec changes are correct, consistent, and well-scoped. The documentation accurately reflects the implementation behavior. The atomic commit is clean. ### Minor Suggestions: - Add a Type/ label to the PR for completeness (e.g., `Type/Task` matching the linked issue). - Consider adding a CHANGELOG entry for consistency, though documentation clarifications are often exempt.
HAL9001 commented 2026-04-28 16:36:14 +00:00
Owner
Copy link

Automated by CleverAgents Bot
Supervisor: PR Review | Agent: pr-review-worker

--- Automated by CleverAgents Bot Supervisor: PR Review | Agent: pr-review-worker
HAL9000 scheduled this pull request to auto merge when all checks succeed 2026-04-28 18:31:50 +00:00
HAL9000 force-pushed feature/issue-4747-agents-validation-attach-synopsis-clarify-key-value from 43e883ea98
All checks were successful
CI / lint (pull_request) Successful in 1m15s
Details
CI / quality (pull_request) Successful in 1m37s
Details
CI / security (pull_request) Successful in 1m43s
Details
CI / typecheck (pull_request) Successful in 1m50s
Details
CI / benchmark-publish (pull_request) Has been skipped
Details
CI / helm (pull_request) Successful in 36s
Details
CI / push-validation (pull_request) Successful in 30s
Details
CI / build (pull_request) Successful in 49s
Details
CI / integration_tests (pull_request) Successful in 3m45s
Details
CI / e2e_tests (pull_request) Successful in 3m40s
Details
CI / unit_tests (pull_request) Successful in 6m32s
Details
CI / docker (pull_request) Successful in 1m44s
Details
CI / coverage (pull_request) Successful in 11m9s
Details
CI / status-check (pull_request) Successful in 4s
Details
to 0ade2526eb
Some checks failed
CI / helm (push) Successful in 30s
Details
CI / benchmark-publish (push) Failing after 44s
Details
CI / build (push) Successful in 51s
Details
CI / lint (push) Successful in 1m15s
Details
CI / quality (push) Successful in 1m20s
Details
CI / push-validation (push) Successful in 35s
Details
CI / typecheck (push) Successful in 1m35s
Details
CI / security (push) Successful in 1m42s
Details
CI / e2e_tests (push) Successful in 4m7s
Details
CI / integration_tests (push) Successful in 4m10s
Details
CI / unit_tests (push) Successful in 5m1s
Details
CI / docker (push) Successful in 1m31s
Details
CI / coverage (push) Successful in 11m28s
Details
CI / status-check (push) Successful in 18s
Details
CI / push-validation (pull_request) Successful in 24s
Details
CI / helm (pull_request) Successful in 30s
Details
CI / lint (pull_request) Successful in 54s
Details
CI / build (pull_request) Successful in 52s
Details
CI / quality (pull_request) Successful in 1m20s
Details
CI / typecheck (pull_request) Successful in 1m24s
Details
CI / security (pull_request) Successful in 1m34s
Details
CI / benchmark-publish (pull_request) Has been skipped
Details
CI / e2e_tests (pull_request) Successful in 4m15s
Details
CI / integration_tests (pull_request) Successful in 5m8s
Details
CI / unit_tests (pull_request) Successful in 6m2s
Details
CI / docker (pull_request) Successful in 1m41s
Details
CI / coverage (pull_request) Failing after 19m6s
Details
CI / status-check (pull_request) Has been cancelled
Details
2026-04-29 17:08:19 +00:00 Compare
HAL9000 merged commit 0ade2526eb into master 2026-04-29 17:27:09 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
HAL9001
Labels
Clear labels   
auto/needs-reevaluation
Controller deferred this PR; awaiting Phase 6+ scope-evaluator or operator re-enablement.

  
controller-managed
Auto-agents controller manages this PR/issue (see tools/controller/deploy/RUNBOOK.md). Remove this label to abandon controller management.

  
auto/blocked-by-deps
PR blocked by an open issue dependency. Operator must close the dep (or remove the dependency link) before the merge driver can act. Auto-cleared by merge_drive when no open deps remain.

  
auto/ci-timeout
Most recent merge cycle hit CI timeout. Driver excludes this PR while last merge_cycle row is < 30 min old; label persists thereafter as visible history.

  
auto/claimed-implementer
Currently being processed by an implementer worker.

  
auto/claimed-merge
Currently being processed by the merge driver.

  
auto/claimed-reviewer
Currently being processed by a reviewer worker.

  
auto/driver-down
Merge driver heartbeat stale; pipeline halted. Closed automatically on next clean tick.

  
auto/invariant-violation
Detected master commit violating the strict merge invariant. Tracked as an issue (not a PR label); kept here for label completeness.

  
auto/last-attempt-tier-0
In-cycle escalation: most recent attempt ran at the Tier 0 slot (`tier-0`). Slot's model defined in .opencode/models/tiers.yaml.

  
auto/last-attempt-tier-1
In-cycle escalation: most recent attempt ran at the Tier 1 slot (`tier-1`). Slot's model defined in .opencode/models/tiers.yaml.

  
auto/last-attempt-tier-2
In-cycle escalation: most recent attempt ran at the Tier 2 slot (`tier-2`). Slot's model defined in .opencode/models/tiers.yaml. Gated behind IMPLEMENTER_ESCALATION_TIER2_ENABLED.

  
auto/last-attempt-tier-min
In-cycle escalation: most recent attempt ran at the Tier -1 slot (`tier-min`). Slot's model defined in .opencode/models/tiers.yaml. Suffix is ``-min`` (not ``--1``) so the Forgejo UI reads naturally.

  
Automation Tracking
Tracking issues used by the AI Automation system for agents to communicate and report.

  
auto/needs-conflict-resolution
Rebase conflict needs LLM conflict-resolver.

  
auto/needs-implementer
Failing CI needs implementer attention.

  
auto/postmortem
Documenting a driver incident or rollback.

  
auto/ready-to-merge
Reviewer has APPROVED this PR and no later REQUEST_CHANGES is outstanding. The merge driver requires this label to even consider a PR for merging. Set by the reviewer worker on APPROVE; cleared on REQUEST_CHANGES.

  
auto/restart-throttled
Train repeatedly lost master-tempo races. Driver excludes via merge_cycle until cooldown elapses; label persists as visible history.

  
auto/revert
Revert PR backing out an invariant violation. Fast-tracked through the merge driver.

  
auto/sentinel
Sentinel PR duplicated from upstream into a personal fork by tools/duplicate_prs_to_fork.py for pipeline testing. Lives only in the fork; the canonical pipeline never sees it.

  
auto/stale-inactivity
No implementer activity for N days. Flagged for human review. Auto-cleared on next push to head branch.

  
auto/unstable
Repeatedly fails on current master (>= 3 ci-fail-on-rebased-sha releases in 12 h). Excluded from driver until human triage.

  
Blocked
A ticket in a blocked state and unable to complete until some other task is completed first.

  
Bounty
$100
 A bounty of $100 for any open-source contributor who provides a MR that solves this issue

  
Bounty
$1000
 A bounty of $1000 for any open-source contributor who provides a MR that solves this issue

  
Bounty
$10000
 A bounty of $10000 for any open-source contributor who provides a MR that solves this issue

  
Bounty
$20
 A bounty of $20 for any open-source contributor who provides a MR that solves this issue

  
Bounty
$2000
 A bounty of $2000 for any open-source contributor who provides a MR that solves this issue

  
Bounty
$250
 A bounty of $250 for any open-source contributor who provides a MR that solves this issue

  
Bounty
$50
 A bounty of $50 for any open-source contributor who provides a MR that solves this issue

  
Bounty
$500
 A bounty of $500 for any open-source contributor who provides a MR that solves this issue

  
Bounty
$5000
 A bounty of $5000 for any open-source contributor who provides a MR that solves this issue

  
Bounty
$750
 A bounty of $750 for any open-source contributor who provides a MR that solves this issue

  
MoSCoW
Could have
 Could have feature in order to satisfy the epic/legendary.

  
MoSCoW
Must have
 Must have feature in order to satisfy the epic/legendary.

  
MoSCoW
Should have
 Should have feature in order to satisfy the epic/legendary.

  
Needs Feedback
There are questions in the ticket that can not be completed until the project owner provides clarity.

  
Points
1
 1 man-hours worth of work for an expert with no learning curve.

  
Points
13
 13 man-hours worth of work for an expert with no learning curve.

  
Points
2
 2 man-hours worth of work for an expert with no learning curve.

  
Points
21
 21 man-hours worth of work for an expert with no learning curve.

  
Points
3
 3 man-hours worth of work for an expert with no learning curve.

  
Points
34
 34 man-hours worth of work for an expert with no learning curve.

  
Points
5
 5 man-hours worth of work for an expert with no learning curve.

  
Points
55
 55 man-hours worth of work for an expert with no learning curve.

  
Points
8
 8 man-hours worth of work for an expert with no learning curve.

  
Points
88
 88 man-hours worth of work for an expert with no learning curve.

  
Priority
Backlog
 This ticket has backlogged priority and is not to be worked on yet

  
Priority
CI Blocker
 Critical priority issue that blocks CI/CD pipeline and prevents PR merges

  
Priority
Critical
 The priority is critical

  
Priority
High
 The priority is high

  
Priority
Low
 The priority is low

  
Priority
Medium
 The priority is medium

  
Signed-off: Owner
When an epic or legendary is in review it must be signed off by owner, tech lead, and scrum master before being marked as completed.

  
Signed-off: Scrum Master
When an epic or legendary is in review it must be signed off by owner, tech lead, and scrum master before being marked as completed.

  
Signed-off: Tech Lead
When an epic or legendary is in review it must be signed off by owner, tech lead, and scrum master before being marked as completed.

  
Spike
A ticket for learning a tool or technology that is needed to be able to do future planning and design.

  
State
Completed
 The ticket has been fully implemented, completed, and merged with the source code. This label should only be applied once a ticket is closed.

  
State
Duplicate
 A ticket that represents the same content as an existing ticket.

  
State
In Progress
 A ticket that is actively being developed.

  
State
In Review
 A ticket that has had some code completed to implement but is waiting to pass peer review and is not yet merged in.

  
State
Paused
 This ticket's work started but wasn't finished. It's on hold (likely in a feature branch) and will be resumed later, either due to a blocker or a delay.

  
State
Unverified
 All new tickets start in this state. A developer may set it to show the ticket is unverified. This means we haven't agreed to work on it. It will either move to a verified state or be closed as wontdo.

  
State
Verified
 The issue has been verified by a developer as legitimate. It will be worked on and verified tickets are now considered part of the backlog.

  
State
Wont Do
 This ticket has been decided it wont be done. This may mean the bug has been determined to not be real (cant verify) or the feature is one we have decided we dont want to adopt.

  
Type
Automation
 Any edits or discussion about the AI automated coding system.

  
Type
Bug
 Something that doesnt work as intended.

  
Type
Discussion
 Anytime a ticket represents a discussion about a subject and doesnt fall into one of the other categories.

  
Type
Documentation
 An error or improvement needed in the documentation.

  
Type
Epic
 Any first tier epic. That is, an epic which contains only issues as children and will not have sub-epics.

  
Type
Feature
 Some new functionality not present.

  
Type
Legendary
 A type of Epic which will contain other Epics.

  
Type
Refactor
 A code change that restructures existing code without changing its external behavior.

  
Type
Support
 Someone needs help using the project.

  
Type
Task
 A generic task that doesnt fit into the other type categories.

  
Type
Testing
 Work exclusively focusing on fixing or expanding testing.

No labels
auto/needs-reevaluation
controller-managed
auto/blocked-by-deps
auto/ci-timeout
auto/claimed-implementer
auto/claimed-merge
auto/claimed-reviewer
auto/driver-down
auto/invariant-violation
auto/last-attempt-tier-0
auto/last-attempt-tier-1
auto/last-attempt-tier-2
auto/last-attempt-tier-min
Automation Tracking
auto/needs-conflict-resolution
auto/needs-implementer
auto/postmortem
auto/ready-to-merge
auto/restart-throttled
auto/revert
auto/sentinel
auto/stale-inactivity
auto/unstable
Blocked
Bounty
$100
Bounty
$1000
Bounty
$10000
Bounty
$20
Bounty
$2000
Bounty
$250
Bounty
$50
Bounty
$500
Bounty
$5000
Bounty
$750
MoSCoW
Could have
MoSCoW
Must have
MoSCoW
Should have
Needs Feedback
Points
1
Points
13
Points
2
Points
21
Points
3
Points
34
Points
5
Points
55
Points
8
Points
88
Priority
Backlog
Priority
CI Blocker
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Signed-off: Owner
Signed-off: Scrum Master
Signed-off: Tech Lead
Spike
State
Completed
State
Duplicate
State
In Progress
State
In Review
State
Paused
State
Unverified
State
Verified
State
Wont Do
Type
Automation
Type
Bug
Type
Discussion
Type
Documentation
Type
Epic
Type
Feature
Type
Legendary
Type
Refactor
Type
Support
Type
Task
Type
Testing
Milestone
Clear milestone
No items
No milestone
v3.5.0
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
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
cleveragents/cleveragents-core!10919
No description provided.