Skip to content

fix(#103): apply documentation label deterministically in post-triage#109

Open
fullsend-ai-coder[bot] wants to merge 1 commit into
mainfrom
agent/103-documentation-label-post-triage
Open

fix(#103): apply documentation label deterministically in post-triage#109
fullsend-ai-coder[bot] wants to merge 1 commit into
mainfrom
agent/103-documentation-label-post-triage

Conversation

@fullsend-ai-coder

Copy link
Copy Markdown
Contributor

Split the documentation case out of the combined documentation|performance branch in the sufficient action handler so that documentation-category issues receive the documentation label, mirroring the bug label fix from PR #96. Add documentation to CONTROL_LABELS to prevent the agent from manipulating it via label_actions.

Test: add sufficient-documentation-gets-documentation-label test case and include documentation in the mock label list.


Closes #103

Post-script verification

  • Branch is not main/master (agent/103-documentation-label-post-triage)
  • Secret scan passed (gitleaks — 8fb64a25b70e4d364f6d535bb252c1bad521629d..HEAD)
  • Pre-commit hooks passed (authoritative run on runner)
  • Tests ran inside sandbox

Split the documentation case out of the combined documentation|performance
branch in the sufficient action handler so that documentation-category
issues receive the documentation label, mirroring the bug label fix
from PR #96. Add documentation to CONTROL_LABELS to prevent the agent
from manipulating it via label_actions.

Test: add sufficient-documentation-gets-documentation-label test case
and include documentation in the mock label list.

Closes #103
@fullsend-ai-coder fullsend-ai-coder Bot requested a review from a team as a code owner July 10, 2026 15:35
@ralphbean ralphbean enabled auto-merge July 10, 2026 19:13
@ralphbean

Copy link
Copy Markdown
Member

/fs-review

@fullsend-ai-review

fullsend-ai-review Bot commented Jul 10, 2026

Copy link
Copy Markdown

🤖 Finished Review · ✅ Success · Started 7:14 PM UTC · Completed 7:25 PM UTC
Commit: b8a817e · View workflow run →

@fullsend-ai-review

Copy link
Copy Markdown

Review

Findings

Medium

Low

  • [incomplete-documentation] docs/triage.md:82 — The control labels list in the example issue-labels skill documentation lists only 5 labels, while the actual CONTROL_LABELS array now has 9 entries. This staleness predates this PR (question, bug, and feature were already missing). The example is a template, not live configuration — enforcement is in post-triage.sh and is correctly updated.

  • [incomplete-documentation] docs/triage.md:40 — The ready-to-code label description does not mention that bug and documentation categories also receive their eponymous labels automatically. This gap predates this PR. Practical impact is minimal as the behavior is enforced by code.

@fullsend-ai-review fullsend-ai-review Bot added the requires-manual-review Review requires human judgment label Jul 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

requires-manual-review Review requires human judgment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Apply documentation label deterministically for documentation-category issues in post-triage.sh

1 participant