Repository navigation
[#482] 🐛 - Dogfooding progress cards remain active after native issue closure - #483
Conversation
|
To preview the documentation for this pull request, visit the following URL:
|
Bugbot: review complete
|
Copilot approval · Human review neededStatus: A protected workflow, policy, or trust-boundary file changed. Technical evidenceRevision: |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## develop #483 +/- ##
===========================================
+ Coverage 93.09% 93.10% +0.01%
===========================================
Files 869 872 +3
Lines 26997 27101 +104
Branches 7810 7846 +36
===========================================
+ Hits 25132 25233 +101
Misses 746 746
- Partials 1119 1122 +3
🚀 New features to boost your workflow:
|
|
/copilot doctor |
|
/copilot sync-branch |
🩺 Diagnóstico del runtime: No disponibleAlgunos requisitos no se pudieron verificar; las comprobaciones completadas siguen visibles. Siguiente acción: Revisa esta ejecución del workflow y vuelve a ejecutar /copilot doctor. Este comando solo lee evidencias. La configuración de aprobación declarada no acredita la salud de los productores ni el acceso de escritura. Comprobaciones
|
…-after-native-issue-closure
Branch synchronized
The fixer resolved 1 conflicted file. 6 verification checks passed. |
|
/copilot recheck |
There was a problem hiding this comment.
Current status: All findings originating in this review are resolved.
Last reconciled one462afe. See aggregate Bugbot status.
🤖 Bugbot review snapshot
Bugbot reported 1 potential problem when commit 0169920 was analyzed. This snapshot is historical; use the status block above for current state. 1 finding is linked to changed code.
Findings
- medium: Progress status publication lacks the required native issue-state query —
src/application/usecases/steps/common/status_card_publication_workflow.ts:106
|
/copilot dismiss "status_card_publication_workflow.ts:106:missing-native-state-query" |
|
1 finding was dismissed. |
|
/copilot sync-branch |
…-after-native-issue-closure
Rama sincronizada
El agente de corrección resolvió 1 archivo con conflictos. Se superaron 6 comprobaciones de verificación. |
|
/copilot recheck |
Closing an issue currently leaves its progress card showing active implementation work. This change archives the same bot-owned card as history and fences late progress jobs against the live issue state, while preserving the reported estimate and the native closure reason.
Validation: local coverage and normal hooks passed 608 suites, 8,641 tests and 35 skipped. The Action-managed Ubuntu synchronization then passed 608 suites, 8,649 tests and 35 skipped after integrating the eight Doctor cases, and completed all six configured checks. The progress capability has 89 runtime cases and 45 budget declarations. Coverage, typecheck, build, lint, documentation and specification validation and normal repository commit hooks passed.
The current review found that escaped archive output could exceed the publication limit. Six regression cases now verify final output length, safe replay and the no-update path. Oversized output leaves the existing card unchanged.
Native merged closure, manual closure, replay and reopening will be checked after promotion.
Closes #482