sofhiaazzhr Claude Opus 4.8 commited on
Commit
e936cfa
Β·
1 Parent(s): 0bec472

[NOTICKET] docs: sync DEV_PLAN status (#8/#9/#20)

Browse files

Mark #8 (trim FE slash menu, KM-678), #9 (verify analysis_id == room_id), and #20 (Help handoff, KM-679) done; resolve the help-endpoint open question (help is a call_agent intent).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Files changed (1) hide show
  1. DEV_PLAN.md +4 -4
DEV_PLAN.md CHANGED
@@ -89,8 +89,8 @@ Status legend: ⬜ not started Β· πŸ”„ in progress Β· βœ… done Β· β›” blocked Β·
89
  | 5 | Report generator β†’ `objective`+`business_questions`, "generated by {user}", formal outline | Sofhia | πŸ”„ pushed (KM-674) | Goal-shape done. Open: author name source (#19); background/EDA/insights mapping |
90
  | 6 | Report skill input contract: `analysis_id` + `user_id` (+ `chat_history`?) | Sofhia/Rifqi | ⬜ | Records-based β‡’ chat_history unused β€” see #17 |
91
  | 7 | `report_id` state update via request to Go, not direct DB | Sofhia + Harry | ⬜ | Needs Go endpoint. See #18 for the rest of the writes |
92
- | 8 | Expose/confirm 4 FE endpoints; comment `check_data` + analysis CRUD | Sofhia | ⬜ | Confirm `help` via `call_agent` (recommended) vs own endpoint |
93
- | 9 | Verify `analysis_id` in `call_agent` contract | Sofhia | πŸ”Ž | Present as `room_id` (== `analysis_id`); document the equivalence |
94
  | 10 | API endpoint doc (MD), 4 endpoints, for Go integration | Rifqi + Sofhia | ⬜ | β€” |
95
  | 11 | Full Python project doc (MD β†’ PDF/Word BRD) | Rifqi | ⬜ | Reuse REPO_STATUS.md as the base |
96
  | 12 | Reconcile/open the `list_tools` PR cleanly (stacked commits) | Rifqi | ⬜ | β€” |
@@ -101,7 +101,7 @@ Status legend: ⬜ not started Β· πŸ”„ in progress Β· βœ… done Β· β›” blocked Β·
101
  | 17 | Reconcile report contract with records-based: remove/flag `chat_history` | Sofhia/Rifqi | ⬜ new | Resolves #5 ↔ #6 contradiction |
102
  | 18 | Confirm Go owns ALL analysis-state writes + both creation gates; make Python `state_store.ensure` read-only | Rifqi + Harry | ⬜ new | Extends #7 to the full decision #4 (G5) |
103
  | 19 | Decide report author display-name source (`users.fullname` vs Go-passed name) | Sofhia | ⬜ new | Raw `user_id` in a formal report looks unprofessional |
104
- | 20 | **Help handoff:** update `handlers/help.py` + `help.md` β€” drop the `problem_validated` tier + `define_problem_statement` action (the skill it points at is gone as of #1) | Sofhia | ⬜ new | Created by #1/#2 (2026-06-25). Help currently still steers users to a removed skill |
105
 
106
  ## 5. Critical path & sequencing
107
 
@@ -114,7 +114,7 @@ Status legend: ⬜ not started Β· πŸ”„ in progress Β· βœ… done Β· β›” blocked Β·
114
  ## 6. Decisions still open (need the team / Harry / lead)
115
 
116
  - `analysis_records`: dedorch-owned vs Python-owned-with-carve-out (#16/#14).
117
- - Whether `help` is its own endpoint or via `call_agent` (#8).
118
  - Author display-name source for the report (#19).
119
  - Confirm Go takes over analysis creation + both creation gates (data-first + mandatory fields) (#18).
120
  - Keep vs drop `chat_history` in the report contract (#17).
 
89
  | 5 | Report generator β†’ `objective`+`business_questions`, "generated by {user}", formal outline | Sofhia | πŸ”„ pushed (KM-674) | Goal-shape done. Open: author name source (#19); background/EDA/insights mapping |
90
  | 6 | Report skill input contract: `analysis_id` + `user_id` (+ `chat_history`?) | Sofhia/Rifqi | ⬜ | Records-based β‡’ chat_history unused β€” see #17 |
91
  | 7 | `report_id` state update via request to Go, not direct DB | Sofhia + Harry | ⬜ | Needs Go endpoint. See #18 for the rest of the writes |
92
+ | 8 | Expose/confirm 4 FE endpoints; comment `check_data` + analysis CRUD | Sofhia | βœ… | KM-678: `list_tools` trimmed to `/help` + `/report` (analytics/check/retrieve commented in the **menu**). `help` confirmed as a `call_agent` intent β€” no own endpoint. Analysis CRUD endpoint left **registered**: "comment the rest" was about the FE slash menu, not killing HTTP routes Go needs |
93
+ | 9 | Verify `analysis_id` in `call_agent` contract | Sofhia | βœ… | Verified: no separate field β€” carried as `room_id` (`analysis_id == room_id`), per REPO_STATUS Β§4/Β§11. Action for Go: send the id as `room_id` |
94
  | 10 | API endpoint doc (MD), 4 endpoints, for Go integration | Rifqi + Sofhia | ⬜ | β€” |
95
  | 11 | Full Python project doc (MD β†’ PDF/Word BRD) | Rifqi | ⬜ | Reuse REPO_STATUS.md as the base |
96
  | 12 | Reconcile/open the `list_tools` PR cleanly (stacked commits) | Rifqi | ⬜ | β€” |
 
101
  | 17 | Reconcile report contract with records-based: remove/flag `chat_history` | Sofhia/Rifqi | ⬜ new | Resolves #5 ↔ #6 contradiction |
102
  | 18 | Confirm Go owns ALL analysis-state writes + both creation gates; make Python `state_store.ensure` read-only | Rifqi + Harry | ⬜ new | Extends #7 to the full decision #4 (G5) |
103
  | 19 | Decide report author display-name source (`users.fullname` vs Go-passed name) | Sofhia | ⬜ new | Raw `user_id` in a formal report looks unprofessional |
104
+ | 20 | **Help handoff:** update `handlers/help.py` + `help.md` β€” drop the `problem_validated` tier + `define_problem_statement` action (the skill it points at is gone as of #1) | Sofhia | βœ… | Done 2026-06-25. `help.py`: actions = `ask_analysis_question` (always) + `generate_report` (if ready); renders objective/business_questions (getattr-tolerant). `help.md` v1β†’v2: 3 tiers, no `/problem_statement`, `/generate report`β†’`/report`. Local test_help updated β†’ 11 pass |
105
 
106
  ## 5. Critical path & sequencing
107
 
 
114
  ## 6. Decisions still open (need the team / Harry / lead)
115
 
116
  - `analysis_records`: dedorch-owned vs Python-owned-with-carve-out (#16/#14).
117
+ - ~~Whether `help` is its own endpoint or via `call_agent` (#8).~~ RESOLVED: `help` is a `call_agent` intent (no own endpoint).
118
  - Author display-name source for the report (#19).
119
  - Confirm Go takes over analysis creation + both creation gates (data-first + mandatory fields) (#18).
120
  - Keep vs drop `chat_history` in the report contract (#17).