docs: diagnose Conversation Agent ownership
This commit is contained in:
@@ -0,0 +1,58 @@
|
||||
# Task: Diagnose Conversation Agent ownership mismatch
|
||||
|
||||
## Identity
|
||||
|
||||
- Task ID: 20260825-conversation-agent-binding-8a4c2e71
|
||||
- Mode: Feature
|
||||
- Branch: codex/20260825-conversation-agent-binding-8a4c2e71-conversation-agent-binding
|
||||
- Worktree: D:\Datas\OthersProjects\makelore-conversation-agent-binding-8a4c2e71
|
||||
- Base commit: 03f96170bb914cd5f8627bc898c9ba5835379531
|
||||
- Owner: codex-root
|
||||
- Status: Done
|
||||
|
||||
## Scope
|
||||
|
||||
- Diagnose why the installed Windows product appears to show a Conversation outside the created Agent.
|
||||
- Verify the active project's persisted Agent/Conversation ownership and trace the Renderer grouping logic.
|
||||
- Produce an evidence-backed explanation only; do not modify product code, user data, or the installed application.
|
||||
|
||||
## Intent And Constraints
|
||||
|
||||
- Distinguish persisted ownership from visual nesting and do not treat a layout expectation as data corruption.
|
||||
- Read only Agent ids/names and Conversation metadata; do not inspect prompts, message bodies, Provider credentials, or secrets.
|
||||
- Do not create subagents. The user requested direct analysis.
|
||||
- Base the diagnosis on cumulative Pi delivery HEAD `03f96170bb914cd5f8627bc898c9ba5835379531` and the installed user's active project catalog.
|
||||
|
||||
## Project Context Loaded
|
||||
|
||||
- Loaded the project-memory startup set and relevant architecture/business-rule indexes.
|
||||
- Reviewed the PI-020 schema-v2, PI-100 core Host API, PI-120 first-Conversation UX, PI-130 feature UI, and latest installed-package hotfix task records.
|
||||
- Audited active task ownership. No product-write overlap exists because this task is read-only; the older partner-create-model-parity task has undefined scope and would require coordination before any later Agent/Conversation implementation change.
|
||||
|
||||
## Plan
|
||||
|
||||
1. Resolve the installed product's active project from the compatibility project catalog.
|
||||
2. Join `.niancode/project.json` Agents to `.niancode/conversations.json` by `agentId` and count missing or ambiguous owners.
|
||||
3. Trace creation validation, persistence, selected-Agent filtering, and Conversation rendering in the current source.
|
||||
4. Run the focused existing UI regression and report confirmed facts, inference, and any remaining hypothesis.
|
||||
|
||||
## Outcome
|
||||
|
||||
- The active installed project is `D:\Datas\GameProjects\testgame` and contains one enabled, unarchived Agent named `ccc`.
|
||||
- Both unarchived Conversations have `agentId=custom-agent-1787571715919-1`, which resolves uniquely to `ccc`; orphan count is zero.
|
||||
- Conversation creation validates the requested Agent and persists its id. Runtime preparation revalidates that the persisted Agent remains available.
|
||||
- The Renderer intentionally uses a master-detail layout: the Agent list and Conversation list are sibling sections, while the Conversation list is filtered to the selected Agent. The apparent lack of nesting is therefore a presentation choice, not an ownership defect.
|
||||
|
||||
## Verification
|
||||
|
||||
- Installed-data ownership join: passed; 2 Conversations, both uniquely owned by `ccc`, 0 orphaned or ambiguous records.
|
||||
- Static source trace: passed; creation, persistence, runtime preparation, selection, and rendering all enforce the same `agentId` relation.
|
||||
- `corepack pnpm exec vitest run tests/unit/coding-chat-panel.test.tsx --maxWorkers=1` on delivered HEAD `03f9617`: passed, 1 file / 15 tests, including Agent-switch Conversation selection isolation.
|
||||
|
||||
## Follow-ups
|
||||
|
||||
- If the product should visually communicate hierarchy, create a separate UX change to nest Conversations under each Agent or label the Conversation section with the selected Agent name. This diagnosis does not authorize that change.
|
||||
|
||||
## Promotion Candidates
|
||||
|
||||
- None recorded.
|
||||
Reference in New Issue
Block a user