Skip to content

t1108: Audit 10 closed issues without PR linkage — all correctly closed#1666

Closed
marcusquinn wants to merge 1 commit intomainfrom
feature/t1108
Closed

t1108: Audit 10 closed issues without PR linkage — all correctly closed#1666
marcusquinn wants to merge 1 commit intomainfrom
feature/t1108

Conversation

@marcusquinn
Copy link
Owner

Summary

Investigated 10 closed GitHub issues flagged by the issue audit as having no PR linkage or completion evidence.

Result: ALL 10 are correctly closed. 0 issues need reopening.

Root Causes

Cause Count Issues
Issue-sync timing (closed before pr: recorded) 2 #1601, #1581
Duplicate issues from dedup_todo_task_ids() bug 7 #1540, #1539, #1538, #1537, #1535, #1534, #1533
No-PR-needed (already complete) 1 #1572

Detailed Findings

Issue Task Verdict Evidence
#1601 t1085.2 FALSE POSITIVE PR #1609 merged
#1581 t1083.3 FALSE POSITIVE PR #1595 merged
#1572 t1079 CORRECTLY CLOSED No PR needed, scripts already compliant
#1540 t1077 (old) DUPLICATE Real task t1068, PR #1546
#1539 t1076 (old) DUPLICATE Real task t1067, PR #1536
#1538 t1075 (old) DUPLICATE Real task t1066, PR #1544
#1537 t1074 (old) DUPLICATE Real task t1065, PR #1543
#1535 t1073 (old) DUPLICATE Real task t1064, PR #1542
#1534 t1072.3 (old) DUPLICATE Real task t1063.3, PR #1530
#1533 t1071.2 (old) DUPLICATE Real task t1063.2, PR #1531

Actions Taken

  • Added Audit Resolution (t1108) comment with evidence to all 10 issues
  • Applied audit:resolved label to all 10 issues
  • No issues reopened (all had legitimate closure reasons)

Recommendations

  1. The dedup bug (root cause for 7/10) was already fixed in t1069 (PR t1069: Fix dedup_todo_task_ids — remove duplicates instead of renaming #1549)
  2. Consider adding a pre-close check to issue-sync that verifies pr: field exists before closing
  3. t1079 in TODO.md should be marked verified:2026-02-18 (no PR needed)

Ref #1656

…ed (t1108)

Investigated 10 GitHub issues flagged as closed without PR linkage:
- 2 false positives: issue-sync closed before pr: field was recorded (#1601, #1581)
- 7 duplicates: created by dedup_todo_task_ids() bug, already fixed in t1069 (#1540, #1539, #1538, #1537, #1535, #1534, #1533)
- 1 no-PR-needed: scripts already had set -euo pipefail (#1572)

Actions: added audit resolution comments and audit:resolved label to all 10 issues.
No issues reopened — all had legitimate closure reasons.
@gemini-code-assist
Copy link

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@github-actions
Copy link

🔍 Code Quality Report

�[0;35m[MONITOR]�[0m Code Review Monitoring Report

�[0;34m[INFO]�[0m Latest Quality Status:
SonarCloud: 0 bugs, 0 vulnerabilities, 24 code smells

�[0;34m[INFO]�[0m Recent monitoring activity:
Wed Feb 18 15:30:14 UTC 2026: Code review monitoring started
Wed Feb 18 15:30:15 UTC 2026: SonarCloud - Bugs: 0, Vulnerabilities: 0, Code Smells: 24

📈 Current Quality Metrics

  • BUGS: 0
  • CODE SMELLS: 24
  • VULNERABILITIES: 0

Generated on: Wed Feb 18 15:30:18 UTC 2026


Generated by AI DevOps Framework Code Review Monitoring

@sonarqubecloud
Copy link

@marcusquinn
Copy link
Owner Author

Closing — task was cancelled.

@marcusquinn marcusquinn deleted the feature/t1108 branch February 21, 2026 01:59
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