Repository navigation
fix(sdd): preserve unpublished ownership through clarification - #885
Conversation
…for-an-explicit-existing-update-request: preserve authorized SDD ownership context across clarification
|
To preview the documentation for this pull request, visit the following URL:
|
There was a problem hiding this comment.
Current status: All findings originating in this review are resolved.
Last reconciled onc2c75b9. See aggregate Bugbot status.
🤖 Bugbot review snapshot
Bugbot reported 1 potential problem when commit 19396c5 was analyzed. This snapshot is historical; use the status block above for current state. 1 finding is linked to changed code.
Findings
- medium: Ownership-question scoring accepts unrelated questions —
quality/sdd/ownership-qualification.cjs:84
Bugbot: review complete
|
Copilot approval · Human review neededStatus: A protected workflow, policy, or trust-boundary file changed. Technical evidenceRevision: |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #885 +/- ##
===========================================
+ Coverage 95.49% 95.50% +0.01%
===========================================
Files 1162 1164 +2
Lines 38326 38354 +28
Branches 11437 11447 +10
===========================================
+ Hits 36598 36629 +31
Misses 665 665
+ Partials 1063 1060 -3
🚀 New features to boost your workflow:
|
…for-an-explicit-existing-update-request: require explicit SDD ownership question assessments
There was a problem hiding this comment.
Current status: All findings originating in this review are resolved.
Last reconciled onc2c75b9. See aggregate Bugbot status.
🤖 Bugbot review snapshot
Bugbot reported 1 potential problem when commit 42db272 was analyzed. This snapshot is historical; use the status block above for current state. 1 finding is linked to changed code.
Findings
- low: Handle undefined responses when scoring ownership questions —
quality/sdd/ownership-qualification.cjs:85
…for-an-explicit-existing-update-request: retain invalid ownership responses without crashing
There was a problem hiding this comment.
Current status: All findings originating in this review are resolved.
Last reconciled onc2c75b9. See aggregate Bugbot status.
🤖 Bugbot review snapshot
Bugbot reported 1 potential problem when commit 4d0810f was analyzed. This snapshot is historical; use the status block above for current state. 1 finding is linked to changed code.
Findings
- medium: Existing issue branch suppresses the unpublished proposal context —
src/application/usecases/sdd/pre_branch_sdd_gate_use_case.ts:185
…for-an-explicit-existing-update-request: separate working branches from publication ownership
Corrige la clasificación entre actualización de un SDD registrado, companion nuevo y capacidad independiente. Tras respuestas autorizadas conserva la propuesta aún no publicada y su ronda, incluso cuando la issue ya tiene una rama de trabajo. Solo la rama enlazada por el registro SDD fija la propiedad publicada; la ruta de un borrador listo sigue usando su fuente correcta. Una aclaración autorizada puede cambiar una propuesta antes de publicar. Se conservan los validadores de propiedad, fuente, autoridad, publicación y diagnóstico seguro.
Closes #856.
Los tres findings de revisión tienen correcciones y regresiones: evaluación semántica explícita de preguntas vinculada al ensayo y respuesta exactos; rechazo de respuestas ausentes antes de consultar preguntas o calcular hashes; y separación entre rama de trabajo y vínculo de publicación SDD. Las propuestas incluyen el significado de su operación registrada y se reconcilian con las respuestas actuales y el catálogo inmutable.
Validación local: 902 suites y 17.704 pruebas de Jest, 35 skips preexistentes y 83 regresiones nuevas; 267 pruebas de extensión con todas las métricas al 100 %. Las 31 líneas modificadas coverables están cubiertas, sin fallos ni parciales, frente a c0e7a13. Tipos, lint, SDD, catálogo, documentación y presupuestos pasan; bundles, paquete de 99 archivos y smoke CLI/API/web pasan. El commit conserva los hooks completos.
La campaña original usa el puerto real del SDK sobre una fuente Git ficticia e inmutable: baseline 19/36, primer candidato 35/36 y revisión 36/36; todos los ensayos y sus fallos se conservan y las preguntas se evalúan explícitamente por el agente, sin atribuir aprobación humana. El suplemento independiente de 15 casos congela los 12 originales y añade tres continuaciones con rama existente. Sus cohortes fueron 14/14/14, 14/15/15 y 15/15/13 sobre 15: las tres se rechazan completas. La versión final obtiene 15/15 en cada uno de tres ensayos, 45/45, cero planes inválidos. No se cambiaron casos, respuestas, modelo, fuente ni criterios entre sus ensayos. La evidencia publica el coordinador usado como texto de auditoría y distingue su preparación del CLI histórico de 12 casos.
Estas mediciones no prueban precisión general ni ejecución completa del consumidor original, cuyos payloads históricos rechazados no están disponibles. El SDD, catálogo y guía explican los contratos, la reproducción y ese alcance. Antes del merge deben pasar los controles del SHA publicado, verificarse explícitamente el tercer finding y quedar todos los hilos resueltos; Codecov oficial debe confirmar 100 % sin parciales frente a la base actual.