flower
/
All briefs
complete draft note flower

Deep Review v2 — single-session & selected-sessions scope

canonical · plan

Spec

markdown

hand-off · dispatch

Dispatch

Auto-dispatch

when it reaches planned

Design-loop

design pass before build

This brief is complete — dispatch is closed.

#46 done fresh flower · flower/102-deep-review-v2
agent: codex 2 scratchpads
You are being dispatched from flower Brief #102: Deep Review v2 — single-session & selected-sessions scope

Recall pointer:
- Use recall_brief with id 102 for the full folder if you need provenance.

Target:
- project: flower (/Users/mikeferrara/Documents/code/flower)
- branch: flower/102-deep-review-v2
- worktree: not specified
- kind: fresh

Current brief spec:
## Goal
Extend the Deep Review feature (v1: #41, project-level) to run against a **single session** or a
**selected group of sessions**, not just an entire project — for when the operator wants a deep,
transcript-verified appraisal of specific sessions rather than the whole project.

## Depends on #41 (v1)
Blocked until Deep Review v1 (#41) ships. Reuses v1 infra: the `deep_review` prompt template (#17
prompt library), DispatchService (high-effort model), the indexed-appraisal output (v1 Q1 decision:
appraisals are indexed into the recall corpus), and child-brief-on-confirm.

## Scope
1. **Session / multi-session selection.** A way to launch a Deep Review scoped to one session id or a
   selected set (from a session list / project session view / recall_active). The dispatched packet
   carries the target session id(s) instead of (or in addition to) a project.
2. **Charter variant.** The `deep_review` template (or a `deep_review.session` variant) instructs the
   agent to pull + verify the specified sessions' raw transcripts (`flower_session_raw`) rather than
   "recent project sessions."
3. **Result capture** identical to v1: appraisal back on the brief + indexed into the recall corpus;
   proposed follow-ups → child briefs on operator confirm.

## Acceptance
- Operator can launch a Deep Review scoped to a single session or a selected group; the agent verifies
  those sessions' raw transcripts and returns an indexed appraisal.
- Reuses v1 dispatch + indexing paths (no divergent second implementation).
- `php artisan test` green + pint. `Brief: #102` trailer.

## Provenance
Chained v2 of #41 (operator, 2026-07-02: "project-level only (v1) - but plan/spec v2/p2 as a follow
up/chained brief"). Speced + chained (depends on #41) by flower-refine (2026-07-03).

Recent/key trace events:
[1] participant_joined flower-refine: (no body)
[2] note_added flower-refine: Chained follow-up to #41 (Deep Review v1, project-level). Extends Deep Review to run against a single session or a selected group of sessions, not just a whole project. Blocked on #41 shipping. Per operator's #41 answer (2026-07-02): "project-level only (v1) - but plan/spec v2/p2 as a follow up/chained brief." Full spec via brief_update_spec.
[3] plan_proposed flower-refine: ## Goal
Extend the Deep Review feature (v1: #41, project-level) to run against a **single session** or a
**selected group of sessions**, not just an entire project — for when the operator wants a deep,
transcript-verified appraisal of specific sessions rather than the whole project.

## Depends on #41 (v1)
Blocked until Deep Review v1 (#41) ships. Reuses v1 infra: the `deep_review` prompt template (#17
prompt library), DispatchService (high-effort model), the indexed-appraisal output (v1 Q1 decision:
appraisals are indexed into the recall corpus), and child-brief-on-confirm.

## Scope
1. **Session / multi-session selection.** A way to launch a Deep Review scoped to one session id or a
   selected set (from a session list / project session view / recall_active). The dispatched packet
   carries the target session id(s) instead of (or in addition to) a project.
2. **Charter variant.** The `deep_review` template (or a `deep_review.session` variant) instructs the
   agent to pull + verify the specified sessions' raw transcripts (`flower_session_raw`) rather than
   "recent project sessions."
3. **Result capture** identical to v1: appraisal back on the brief + indexed into the recall corpus;
   proposed follow-ups → child briefs on operator confirm.

## Acceptance
- Operator can launch a Deep Review scoped to a single session or a selected group; the agent verifies
  those sessions' raw transcripts and returns an indexed appraisal.
- Reuses v1 dispatch + indexing paths (no divergent second implementation).
- `php artisan test` green + pint. `Brief: #102` trailer.

## Provenance
Chained v2 of #41 (operator, 2026-07-02: "project-level only (v1) - but plan/spec v2/p2 as a follow
up/chained brief"). Speced + chained (depends on #41) by flower-refine (2026-07-03).
[4] dependency_added flower-refine: Now depends on #41 (What do you think about something like a 'deep review' of a project or).
[5] status_change flower-refine: (no body)
[6] link_added flower-refine: (no body)
[7] link_added flower-refine: (no body)
[8] participant_joined flower-orchestrator: (no body)
[9] unblocked flower-orchestrator: Unblocked — #41 reached complete.

Recommended linked context:
{
    "todos": [],
    "scratchpads": [
        {
            "id": 364,
            "solo_scratchpad_id": "1055",
            "name": "flower-refine — reset handoff (2026-07-03)",
            "archived": false,
            "revision": 1
        },
        {
            "id": 375,
            "solo_scratchpad_id": "1067",
            "name": "flower-refine — reset handoff (2026-07-03 #2)",
            "archived": false,
            "revision": 2
        }
    ]
}

Execution notes:
- Treat the brief as the source of truth.
- Keep work scoped to this dispatch request.
- Use brief_append / brief_update_status when reporting material progress; as your final dispatched-worker step, call brief_dispatch_complete with dispatch_request_id (or brief_id) and actor_ref.
- Codex workers should verify mutating Flower tools with tool_search query `brief_append brief_dispatch_complete flower_feedback` (limit 20) when tool availability is in doubt; report raw SEE/LOAD vs NOT visible instead of silently using local fallbacks.
- Add a git commit trailer `Brief: #102` to every commit for this brief so flower can exact-link commits back to the brief.

provenance · append-only

Trace

live
or paste a screenshot uploading…
  1. link added 2d ago
    agent · system:commit-trailer
  2. participant joined 2d ago
    system · system:commit-trailer
  3. merged 2d ago

    Merged flower/102-deep-review-v2 → master as merge commit 1627042 (worker commit bf56dc3). Post-merge on MAIN: migration 2026_07_03_153000_seed_deep_review_session_prompt_template applied + PromptTemplateSeeder seeded; full suite green (726 tests, 724 passed, 2 skipped, 4931 assertions); pint clean. Deep Review v2 (session-scoped) is live on MAIN. Merged by flower-orchestrator (daemon 16).

    agent · flower-orchestrator
  4. status change 2d ago
    agent · flower-102-deepreview-v2
  5. dispatched 2d ago

    Dispatch request #46 marked done.

    agent · flower-102-deepreview-v2
  6. note added 2d ago

    Completed implementation and committed bf56dc39bf5f244c8500cc7c74bb7d609291eee5 on branch flower/102-deep-review-v2. Verification: focused DeepReviewTest, DispatchServiceTest, Sessions IndexTest, and Prompts IndexTest passed; full ~/bin/php artisan test passed (726 tests, 724 passed, 2 skipped); ./vendor/bin/pint passed. Worktree is clean.

    agent · flower-102-deepreview-v2
  7. note added 2d ago

    Implemented the v2 path in the existing DeepReviewService/DispatchService flow: added a keyed deep_review.session.default charter, session-scoped create+dispatch with seed session links, optional review_session_ids in dispatch recommended context, and a Sessions page selected-session launcher. Focused tests pass for DeepReviewTest, DispatchServiceTest, tests/Feature/Sessions/IndexTest.php, and tests/Feature/Prompts/IndexTest.php.

    agent · flower-102-deepreview-v2
  8. note added 2d ago

    Started implementation in Solo process 1043 / project 53. Confirmed #102 scope and #41 reference spec, verified required MCP tools visible via tool_search, and checked out branch flower/102-deep-review-v2 from master in the flower-backend worktree.

    agent · flower-102-deepreview-v2
  9. participant joined 2d ago
    system · flower-102-deepreview-v2
  10. dispatched 2d ago

    Dispatch request #46 queued for flower.

    agent · flower-orchestrator
  11. status change 2d ago
    agent · flower-orchestrator
  12. unblocked 2d ago

    Unblocked — #41 reached complete.

    system · flower-orchestrator
  13. participant joined 2d ago
    system · flower-orchestrator
  14. link added 2d ago
    agent · flower-refine
  15. link added 2d ago
    agent · flower-refine
  16. status change 2d ago
    agent · flower-refine
  17. dependency added 2d ago

    Now depends on #41 (What do you think about something like a 'deep review' of a project or).

    agent · flower-refine
  18. plan proposed 2d ago

    ## Goal Extend the Deep Review feature (v1: #41, project-level) to run against a **single session** or a **selected group of sessions**, not just an entire project — for when the operator wants a deep, transcript-verified appraisal of specific sessions rather than the whole project. ## Depends on #41 (v1) Blocked until Deep Review v1 (#41) ships. Reuses v1 infra: the `deep_review` prompt template (#17 prompt library), DispatchService (high-effort model), the indexed-appraisal output (v1 Q1 decision: appraisals are indexed into the recall corpus), and child-brief-on-confirm. ## Scope 1. **Session / multi-session selection.** A way to launch a Deep Review scoped to one session id or a selected set (from a session list / project session view / recall_active). The dispatched packet carries the target session id(s) instead of (or in addition to) a project. 2. **Charter variant.** The `deep_review` template (or a `deep_review.session` variant) instructs the agent to pull + verify the specified sessions' raw transcripts (`flower_session_raw`) rather than "recent project sessions." 3. **Result capture** identical to v1: appraisal back on the brief + indexed into the recall corpus; proposed follow-ups → child briefs on operator confirm. ## Acceptance - Operator can launch a Deep Review scoped to a single session or a selected group; the agent verifies those sessions' raw transcripts and returns an indexed appraisal. - Reuses v1 dispatch + indexing paths (no divergent second implementation). - `php artisan test` green + pint. `Brief: #102` trailer. ## Provenance Chained v2 of #41 (operator, 2026-07-02: "project-level only (v1) - but plan/spec v2/p2 as a follow up/chained brief"). Speced + chained (depends on #41) by flower-refine (2026-07-03).

    agent · flower-refine
  19. note added 2d ago

    Chained follow-up to #41 (Deep Review v1, project-level). Extends Deep Review to run against a single session or a selected group of sessions, not just a whole project. Blocked on #41 shipping. Per operator's #41 answer (2026-07-02): "project-level only (v1) - but plan/spec v2/p2 as a follow up/chained brief." Full spec via brief_update_spec.

    agent · flower-refine
  20. participant joined 2d ago
    system · flower-refine

epic · dependencies

Relationships

epic parent

depends on

agents · waves

Participants

  • flower-refine participant · active
  • flower-orchestrator participant · active
  • flower-102-deepreview-v2 participant · active
  • system:commit-trailer participant · active

trace · graph

Links

  • Commit #1712 execution
  • Scratchpad #375 execution
  • Scratchpad #364 execution

scope

Projects

  • flower · primary

dogfood · read-only

Agent’s-eye view

The literal recall_brief payload an agent gets — same service path as the MCP tool.