Add retroactive task specs for ad-hoc changes
This commit is contained in:
@@ -0,0 +1,20 @@
|
||||
# SPEC: Show Artifact Badges on Pending Review Tasks
|
||||
|
||||
## Overview
|
||||
|
||||
Tasks pending review don't show what artifacts were produced. Card should display compact artifact badges (SPEC.md, DESIGN.md, etc.) so reviewers know what needs review at a glance.
|
||||
|
||||
## Changes
|
||||
|
||||
- Add `ARTIFACT_LABELS` mapping
|
||||
- Show artifact badges on cards when review status is pending or changes_requested
|
||||
- CSS for `.task-card-artifacts` and `.artifact-badge`
|
||||
|
||||
## Acceptance Criteria
|
||||
|
||||
- [ ] Pending review tasks show artifact badge chips on their card
|
||||
- [ ] Approved/completed tasks do not show artifact badges
|
||||
|
||||
## VERDICT
|
||||
|
||||
PASS — implemented in commit 75cb4a1
|
||||
@@ -0,0 +1,19 @@
|
||||
# SPEC: Display SPEC.md Content in Task Detail Panel
|
||||
|
||||
## Overview
|
||||
|
||||
The task detail panel shows verdict and bug report content but not the task's SPEC.md. Users need to see the specification when reviewing a task.
|
||||
|
||||
## Changes
|
||||
|
||||
- Add `spec_content` field to API responses (`_serve_tasks`, `_serve_task`)
|
||||
- Display SPEC.md content in the detail panel under a "Specification" section
|
||||
|
||||
## Acceptance Criteria
|
||||
|
||||
- [ ] SPEC.md content appears in detail panel
|
||||
- [ ] API serves `spec_content` for all tasks
|
||||
|
||||
## VERDICT
|
||||
|
||||
PASS — implemented in commit 01d8451
|
||||
@@ -0,0 +1,24 @@
|
||||
# SPEC: Convert Task Detail Panel to Centered Modal
|
||||
|
||||
## Overview
|
||||
|
||||
The sliding side panel (380px wide) is too narrow to read SPEC.md, verdicts, and other task content. Replace it with a centered modal overlay that uses the full screen width.
|
||||
|
||||
## Changes
|
||||
|
||||
- Replace `.detail-panel` fixed sidebar with `.modal-overlay` + `.modal-panel`
|
||||
- Wrapped detail panel HTML in a modal overlay container
|
||||
- Added backdrop click to close
|
||||
- Updated JS references from `detail-panel` to `detail-overlay`
|
||||
|
||||
## Acceptance Criteria
|
||||
|
||||
- [ ] Modal is centered with 720px max width, 85vh max height
|
||||
- [ ] Content is scrollable within the modal
|
||||
- [ ] Clicking backdrop closes the modal
|
||||
- [ ] Escape key closes the modal
|
||||
- [ ] Existing functionality (review buttons, artifact display) is preserved
|
||||
|
||||
## VERDICT
|
||||
|
||||
PASS — implemented in commit 6630577
|
||||
Reference in New Issue
Block a user