docs: integrate design stream and quote handoff
This commit is contained in:
@@ -0,0 +1,114 @@
|
||||
# Task: Integrate AI Design streamed replies and quote handoff
|
||||
|
||||
## Identity
|
||||
|
||||
- Task ID: 20260904-integrate-design-stream-5e7c2a91
|
||||
- Mode: Integration
|
||||
- Branch: codex/20260904-diagnose-design-stream-6a4e9c21-diagnose-design-stream
|
||||
- Worktree: D:\Datas\OthersProjects\.codex-worktrees\makelore\20260904-diagnose-design-stream-6a4e9c21
|
||||
- Base commit: 23f96a523eb37d8397bb3766ce95a59d56e59225
|
||||
- Owner: codex-root
|
||||
- Status: Ready for Integration
|
||||
|
||||
## Scope
|
||||
|
||||
- Integrate diagnosis commit `229b1b1ce39b7f1541a93ea3c980ab82b6d6266c`
|
||||
and implementation commit `23f96a523eb37d8397bb3766ce95a59d56e59225`
|
||||
from tasks `20260904-diagnose-design-stream-6a4e9c21` and
|
||||
`20260904-fix-design-stream-4f7b91c2` onto local client `main`.
|
||||
- Reconcile the accepted promotion candidate into `current-state.md`, the AI Design
|
||||
architecture boundary, and business rules: a delta may appear only as one visibly
|
||||
unfinished assistant reply; Quote review remains separate from paid generation
|
||||
confirmation.
|
||||
- Preserve the three known foreign untracked task records in the root checkout exactly
|
||||
as found. Do not modify the Works Square server, billing, Plugin navigation,
|
||||
packaging, publication, remote refs, or any unrelated client feature.
|
||||
|
||||
## Intent And Constraints
|
||||
|
||||
- The user explicitly requested this completed client fix be merged to the main branch
|
||||
and had already confirmed the product decision behind the canonical wording change.
|
||||
- Root `main` was verified at exact head
|
||||
`aa3f52a8f8414c9ad939d396bd2828c3e07c70e7`, with only the three previously known
|
||||
untracked records. The source is a linear descendant of that frontier, so use a
|
||||
fast-forward after canonical reconciliation; do not stash, reset, clean, add, or
|
||||
adopt the foreign records.
|
||||
- Keep `workspace.turns` as canonical conversation history. The provisional stream is
|
||||
presentation tied to an existing pending chat identity, never a second completed
|
||||
turn or generic progress banner.
|
||||
- Keep production authorization two-step: Quote request/reveal may be prompted by the
|
||||
conversation, while only explicit confirmation of the immutable Quote ID creates a
|
||||
paid Task.
|
||||
|
||||
## Project Context Loaded
|
||||
|
||||
- Concurrent Task Gate passed for integration task
|
||||
`20260904-integrate-design-stream-5e7c2a91`, owner `codex-root`, mode Integration,
|
||||
branch `codex/20260904-diagnose-design-stream-6a4e9c21-diagnose-design-stream`,
|
||||
worktree
|
||||
`D:\Datas\OthersProjects\.codex-worktrees\makelore\20260904-diagnose-design-stream-6a4e9c21`,
|
||||
base `23f96a523eb37d8397bb3766ce95a59d56e59225`; the repository Integration lock is held.
|
||||
- Loaded the startup memory set, current integrated state, system overview, business
|
||||
rules, ADR-007 context, and both completed source task records including outcome,
|
||||
verification, follow-up, and promotion candidates.
|
||||
- Relevant active peers are unrelated historical/stale task records. The exclusive
|
||||
Integration lock prevents concurrent canonical reconciliation; no peer owns a
|
||||
conflicting current AI Design product decision.
|
||||
- The old canonical rule hid raw delta text entirely. The accepted source narrows that
|
||||
rule without weakening authority: exactly one provisional bubble may paint the
|
||||
already validated assistant text, while canonical turns, Current Specification,
|
||||
stable operation identity, Quote identity, and explicit payment confirmation remain
|
||||
unchanged. The user's confirmation resolves the semantic conflict.
|
||||
- Planning Gate result: Passed.
|
||||
|
||||
## Plan
|
||||
|
||||
1. Reconcile the accepted AI Design stream/Quote behavior into canonical architecture,
|
||||
business rules, and current-state integration evidence.
|
||||
2. Re-run task-aware document and source verification at the final integration head.
|
||||
3. Commit the integration record, fast-forward exact root `main`, verify the three
|
||||
foreign untracked files are unchanged, then complete and release the Integration
|
||||
lock.
|
||||
|
||||
## Outcome
|
||||
|
||||
- Accepted and reconciled both source tasks on top of exact local `main` frontier
|
||||
`aa3f52a8f8414c9ad939d396bd2828c3e07c70e7`; their commit chain is linear and
|
||||
contains no unrelated product changes.
|
||||
- Updated the canonical architecture and business rules to distinguish one
|
||||
operation-bound unfinished assistant bubble from canonical conversation history
|
||||
and from the prohibited generic progress banner. Unknown operations retain their
|
||||
identity/partial text and cannot be erased by unrelated updates.
|
||||
- Recorded the ready-conversation Quote handoff: desktop reveals the offered Quote,
|
||||
mobile opens it, and neither path creates a Task until the user explicitly confirms
|
||||
the immutable Quote ID. Unknown Quote requests keep and retry the original operation
|
||||
instead of creating another intent.
|
||||
- Updated `current-state.md` with source commits, behavior, verification, the known
|
||||
post-validation streaming limitation, and the explicit no-server/no-billing scope.
|
||||
- Prepared the integrated branch for a fast-forward of root `main`; the root's three
|
||||
pre-existing untracked task records remain outside this commit and are not adopted,
|
||||
staged, modified, moved, or deleted.
|
||||
|
||||
## Verification
|
||||
|
||||
- Source task evidence: focused 35/35, Electron Quote-confirm-Task 1/1, typecheck,
|
||||
scoped ESLint, Vite build, diff check, pressure 1/1, and independent final review
|
||||
passed. The full suite's only miss was an unrelated Pi process 2-second timing check,
|
||||
which passed immediately in isolation.
|
||||
- Integration-head rerun: focused Renderer/store/Card suite 3 files / 35 tests passed;
|
||||
`pnpm exec tsc --noEmit` passed.
|
||||
- Integration-head Electron flow passed 1/1 and again exercised the conversation CTA,
|
||||
Quote visibility, explicit 12-Design-Point confirmation, and queued Task projection.
|
||||
- `git diff --check`, `check_project_docs.py`, and task-aware
|
||||
`check_doc_drift.py --task-id 20260904-integrate-design-stream-5e7c2a91` passed.
|
||||
|
||||
## Follow-ups
|
||||
|
||||
- Provider-native first-token streaming still requires a separately scoped server
|
||||
contract; this integration only exposes the existing validated-message chunk
|
||||
projection honestly.
|
||||
|
||||
## Promotion Candidates
|
||||
|
||||
- None. The source candidates were accepted by the user and applied to canonical
|
||||
architecture, domain rules, and current state in this integration task.
|
||||
Reference in New Issue
Block a user