454 lines
56 KiB
Markdown
454 lines
56 KiB
Markdown
# Task: 拉取远程主分支、合并并推送
|
|
|
|
## Identity
|
|
|
|
- Task ID: 20260813-sync-push-main-9c2f71
|
|
- Mode: Integration
|
|
- Branch: main
|
|
- Worktree: D:\Datas\OthersProjects\makelore
|
|
- Base commit: 4fbd1d3b24493532ce479729da2c5e5e1709c8a9
|
|
- Owner: codex
|
|
- Status: Blocked
|
|
|
|
## Scope
|
|
|
|
- On 2026-08-16, resume the existing Integration owner to merge reviewed cross-platform in-app Robot hotspot connection source `c1326a2` into local `main`, supersede only ADR-002's manual operating-system hotspot-selection step, preserve firmware/credential/Binding boundaries, and keep remote push outside this resumption.
|
|
- On 2026-08-16, resume the existing Integration owner to merge reviewed default-on Guided Hotspot Binding source `b78fc07` into local `main`, accept the user's explicit reversal of the prior default-off policy, preserve exact environment value `0` as rollback, and keep firmware edits and remote push outside this resumption.
|
|
- On 2026-08-16, resume the existing Integration owner to merge reviewed Robot Guided Hotspot Binding implementation commit `b7a1590` into local `main`, reconcile canonical memory from planned to implemented/default-off, and keep firmware edits, capability enablement, and remote push outside this resumption.
|
|
- On 2026-08-16, resume the existing Integration owner to accept reviewed Robot Guided Hotspot Binding V1 design commit `14afe4a`, promote only its confirmed minimal-firmware decision into canonical memory, and keep product implementation, firmware changes, and remote push outside this integration step.
|
|
- On 2026-08-16, resume the existing Integration owner to merge reviewed Robot configuration-catalog/editor source `fe55dee` into local `main`, promote the accepted dynamic catalog boundary, and keep remote push outside this resumption.
|
|
- On 2026-08-15, resume the existing Integration owner to merge reviewed Robot configuration-schema fix `ea75b06` into local `main`, accepting canonical weak numeric response ETags produced by the deployed compression layer while keeping strong `If-Match` writes.
|
|
- On 2026-08-15, resume the existing Integration owner to merge reviewed Robot configuration-loading fix `1bcd519` into local `main`, preserving the page error/retry state and Main response-body deadline.
|
|
- On 2026-08-15, resume the existing Integration owner to merge reviewed source commit `fd9b5b46a913c515e94e4e26f185d43866c2581f` for the Codex-style AI Programming context-compaction timeline into local `main` and promote its accepted lifecycle facts.
|
|
- On 2026-08-14, resume the existing Integration owner to merge reviewed AI Canvas source commit `22378efcee07e7fb80b651e65e3202f1a1dfea1d` into local `main` and promote its accepted project-memory candidates.
|
|
- Resume the existing integration owner to fast-forward the completed Robot / AI hardware source commit into local `main` after verifying the current remote `main` tip.
|
|
- Fetch the authoritative remote `main`, inspect all commits not present in the
|
|
local branch, and merge them into the reviewed local `main` without rewriting
|
|
either history.
|
|
- Resolve any code or project-memory conflicts semantically, re-run verification
|
|
proportionate to the fetched changes, obtain an independent final review, and
|
|
push the resulting `main` to its configured remote.
|
|
- Record the exact fetched remote tip, merge topology, verification, and push
|
|
result in this task record.
|
|
|
|
## Intent And Constraints
|
|
|
|
- Do not rebase, reset, stash, force-push, or discard either local or remote
|
|
commits. The user authorized a normal pull/merge/push workflow.
|
|
- Treat the current `origin/main` value as a stale local tracking reference until
|
|
`git fetch origin main` succeeds.
|
|
- Preserve the already reviewed Updater and image-to-image behavior and the
|
|
repository's Main-owned security boundaries.
|
|
- Source/peer task records remain read-only. This integration task may update
|
|
only its own task record and any canonical documents that genuinely require
|
|
semantic reconciliation.
|
|
- Push only after the merge result is clean, verified, and independently
|
|
reviewed. Never use `--force`.
|
|
- The 2026-08-14 user request authorizes the local `main` merge only. It does not expand this resumption into a remote push; the existing authentication blocker remains a separate follow-up.
|
|
- The 2026-08-15 context-compaction request likewise authorizes only a local `main` merge. It does not authorize a remote push or changing the existing OpenCode/model compaction threshold.
|
|
- The 2026-08-16 Robot onboarding confirmation accepts the current-firmware Hotspot + six-digit Binding V1. It does not authorize firmware edits, claim automatic nearby discovery, enable the pilot capability by default, or revive the unready Security 2/automatic-claim proposal as a V1 contract.
|
|
- The reviewed implementation may move canonical truth from planned to present, but release guidance must retain the exact shipped-firmware, issuer/validator, gate-on Electron, and physical-device smoke prerequisites. The source task record remains read-only and must stay on its feature history.
|
|
- The user's latest instruction explicitly authorizes default-on and supersedes only the earlier default-off/capability-not-enabled constraint. It does not authorize firmware changes, automatic discovery/claim claims, arbitrary portal URLs, Wi-Fi credential handling, or remote push. Missing installed-Electron/physical-device evidence remains an explicit residual release risk, not completed evidence.
|
|
- The user has now explicitly authorized page-owned selection and connection of nearby open `Xiaozhi-*` provisioning hotspots on both Windows and macOS. This supersedes only manual operating-system hotspot selection; discovery remains unauthenticated convenience, Main remains the sole native-network owner, and signed macOS plus physical-Robot smoke remain release gates.
|
|
|
|
## Project Context Loaded
|
|
|
|
Task context:
|
|
- Task ID: `20260813-sync-push-main-9c2f71`
|
|
- Mode: Integration
|
|
- Branch: `main`
|
|
- Worktree: `D:\Datas\OthersProjects\makelore`
|
|
- Base commit: `4fbd1d3b24493532ce479729da2c5e5e1709c8a9`
|
|
- Other active local tasks: all are isolated in separate registered worktrees;
|
|
their Scope, Intent And Constraints, and Promotion Candidates were reviewed.
|
|
- Overlap or semantic-conflict assessment: no peer owns this worktree or has an
|
|
unresolved decision that changes this synchronization plan. Remote overlap
|
|
remains unknown until fetch and will be inspected before merge.
|
|
|
|
Read:
|
|
- `.project-docs/05-agent-entry/read-before-planning.md`
|
|
- `.project-docs/05-agent-entry/memory-index.md`
|
|
- this active task record
|
|
- `.project-docs/00-brief/project-positioning.md`
|
|
- `.project-docs/30-worklog/current-state.md`
|
|
- `.project-docs/10-decisions/decision-index.md`
|
|
- `.project-docs/20-architecture/system-overview.md`
|
|
- `.project-docs/05-agent-entry/integration-gate.md`
|
|
- `.project-docs/50-evidence/evidence-index.md`
|
|
- `.project-docs/60-reflection/reflection-index.md`
|
|
- `.project-docs/80-commitments/commitments.md`
|
|
- `.project-docs/90-maintenance/stale-items.md`
|
|
- all registry-owned peer task records through the required planning sections
|
|
|
|
Relevant understanding:
|
|
- Project goal: keep the reviewed local Makelore client history while
|
|
incorporating legitimate remote `main` work and publishing one coherent tip.
|
|
- Current integrated focus: local `main` already contains the reviewed Updater
|
|
and single-reference image-to-image changes; external production smoke remains
|
|
pending and must not be overclaimed by this Git synchronization.
|
|
- Active constraints: preserve Main-owned updater/workspace boundaries, existing
|
|
canonical decisions, and source task records; push must be non-forced.
|
|
- Evidence and commitments: the local tip passed 84 focused tests, typecheck,
|
|
scoped lint, `build:vite`, project-document gates, and independent Sol review;
|
|
three full-suite failures match the documented pre-existing baseline.
|
|
- Unknowns: the latest remote tip and its semantic overlap are not verified
|
|
until fetch.
|
|
|
|
Gate result:
|
|
- Passed.
|
|
|
|
### 2026-08-16 Cross-Platform Robot Hotspot Integration Resume
|
|
|
|
- Reused the existing Integration owner because it exclusively owns clean local `main` at `abecd5f34485ab467e5f032c618083d88e34b74d`; `task_context.py touch` refreshed the reservation and the registry reports this exact `main` worktree/branch owner.
|
|
- Formed reviewed source commit `c1326a298026a697181c822cdd3062cc96c3aa2d`; it is exactly one commit ahead of current `main`, its feature task is `ready_for_integration`, its worktree is clean, and final read-only Sol review returned PASS on Standards and Spec with no P0-P3 findings.
|
|
- Read the source task outcome, verification, follow-ups, and proposal against ADR-002, current Robot architecture/domain memory, release commitments, and relevant peer task scopes. The user-confirmed Windows/macOS page connection decision resolves the only semantic conflict with ADR-002's manual operating-system hotspot-selection step.
|
|
- No peer owns `main`. Related Robot tasks are isolated and either support this direction or remain historical/deferred: authenticated automatic claim is not revived, discovery does not prove identity, and source task records remain read-only.
|
|
- Gate result: Passed for a local no-ff merge and canonical reconciliation. Firmware edits, remote push, signed macOS x64/arm64 acceptance, and physical-Robot acceptance remain outside this resumption.
|
|
|
|
### 2026-08-16 Guided Hotspot Default-On Integration Resume
|
|
|
|
- Reused the existing Integration owner because it still exclusively owns clean local `main` at `971865c256c2ddfe1f8ab4f8f4731a1f608599c0`; `task_context.py touch` refreshed the reservation and the registry reports this exact `main` worktree/branch owner.
|
|
- Verified feature task `20260816-enable-guided-hotspot-4e91c2` is `ready_for_integration`, reviewed source commit `b78fc07` is based exactly on current `main`, and independent final Sol review returned PASS with no P0-P3 findings.
|
|
- Read the source outcome, verification, follow-ups, and promotion candidate against ADR-002 and current canonical Robot memory. The source changes only the Main default and focused route tests; firmware, Renderer, Host wire shape, fixed portal, cloud Binding, and credential boundaries remain unchanged.
|
|
- The current user decision resolves the sole semantic conflict: Guided Hotspot Binding is now default-on, with exact environment value `0` as rollback. The exact shipped firmware/issuer and real native-opener/physical-device evidence remain unverified and must not be claimed as passed.
|
|
- Gate result: Passed for local merge and canonical policy reconciliation. Remote push remains outside this resumption.
|
|
|
|
### 2026-08-14 AI Canvas Integration Resume
|
|
|
|
- Reused the same Integration owner because it still exclusively owns `main` and the repository integration lock; `task_context.py touch` refreshed the existing reservation.
|
|
- Verified the main worktree was clean at `88f9ee8708beeb1ab45ca741807f6cca0f075ac9` before merging.
|
|
- `git fetch origin main --prune` confirmed `origin/main` is also `88f9ee8708beeb1ab45ca741807f6cca0f075ac9`.
|
|
- Source `22378efcee07e7fb80b651e65e3202f1a1dfea1d` is exactly one commit ahead of `main`, with `88f9ee8` as merge base; no remote-only or unrelated main commit exists.
|
|
- Read the source task, ADR-001, canonical AI Canvas architecture/domain memory, and all registry-owned peer task Scope / Intent / Promotion Candidate sections. The source preserves Workspace-owned tasks and Conversation-owned state; no semantic conflict requires human resolution.
|
|
- Gate result: Passed for the local AI Canvas merge. Remote push remains outside this resumption.
|
|
|
|
### 2026-08-15 Context Compaction Integration Resume
|
|
|
|
- Reused the existing Integration owner because it still exclusively owns `main` and the repository integration lock; `task_context.py touch` refreshed the reservation.
|
|
- Verified the main worktree was clean at `953b0f491e44036bc8273fb29dd880e38defae24`, with local `main` two commits ahead of the locally tracked `origin/main`.
|
|
- Source task `20260814-codex-compaction-ux-74b3f2a1` was `ready_for_integration`, its final Sol review returned PASS, and source commit `fd9b5b46a913c515e94e4e26f185d43866c2581f` is exactly one direct descendant of current `main`.
|
|
- Read the active integration record, source task outcome/verification/promotion candidate, canonical AI Programming architecture/current-state memory, and every registered peer task Scope / Intent / Promotion Candidate section.
|
|
- The related threshold and UI-audit tasks agree with the source: automatic compaction remains OpenCode/model-limit-owned, while this source changes only the persistent Renderer interaction and run lifecycle. No semantic conflict requires human resolution.
|
|
- Gate result: Passed for the local context-compaction merge. Remote push remains outside this resumption.
|
|
|
|
### 2026-08-15 Robot Configuration Loading Integration Resume
|
|
|
|
- Reused this Integration owner because it still exclusively owns `main` and the repository integration lock; `task_context.py touch` refreshed the reservation.
|
|
- Verified source task `20260815-robot-config-loading-4f8c2d` is ready for integration, its final Sol review returned PASS, and source commit `1bcd519` is a direct descendant of current `main` at merge base `a4050f0`.
|
|
- A fresh `git fetch origin main --prune` succeeded and confirmed local `main` and `origin/main` are both exactly `a4050f0`, with no remote-only commit or divergence.
|
|
- The source changes only the Robot page, Main-owned AI hardware route, focused tests, and its own source task record. It preserves Main-owned auth/idempotency/ETag and introduces no semantic overlap with registered peer work.
|
|
- Gate result: Passed for the local Robot configuration-loading merge. This request authorizes a local `main` merge, not an automatic remote push.
|
|
|
|
### 2026-08-15 Robot Configuration Schema Integration Resume
|
|
|
|
- Reused this Integration owner because it still exclusively owns `main` and the repository integration lock; `task_context.py touch` refreshed the existing reservation.
|
|
- Verified source task `20260815-robot-config-schema-8e2c41` is ready for integration, source commit `ea75b06` is a direct descendant of current `main` at `a26a53a`, and its final read-only Sol review returned PASS.
|
|
- Production evidence distinguishes the hops: Xiaozhi-to-Works returns a canonical strong numeric ETag, while the compressed public Works response was observed with matching canonical weak `W/\"0\"`, numeric `config_revision: 0`, and the exact expected key set.
|
|
- The source changes only Electron Main response-revision parsing and its focused route tests. It still requires DTO/revision equality and emits only strong `If-Match` for writes; no authentication, idempotency, response-bound, or redaction boundary changes.
|
|
- Gate result: Passed for the local Robot configuration-schema merge. Remote push remains outside this resumption.
|
|
|
|
### 2026-08-16 Robot Guided Hotspot Binding Implementation Integration Resume
|
|
|
|
- Reused the existing Integration owner because it exclusively owns clean local `main` at `54443232dd6a07dc1f3df5b33df00f665540497a`; `task_context.py touch` refreshed the reservation and registry doctor passed.
|
|
- Verified source task `20260816-guided-hotspot-binding-7c4d2e` is `ready_for_integration`, source commit `b7a1590ca132c971ffff177cb1e2aa47f96358cf` is a direct descendant of current `main`, its worktree is clean, and independent Sol re-review returned PASS with no P0-P3 findings.
|
|
- Read the source task outcome, verification, follow-ups, and promotion candidate against ADR-002 and current canonical Robot memory. The implementation preserves the accepted zero-firmware/default-off boundary; its only promotion candidate remains deferred until release-gate evidence exists.
|
|
- No registered peer owns `main` or contradicts this implementation. The prior Security 2/automatic-claim proposal remains a deferred future direction and is not part of this merge.
|
|
- Gate result: Passed for the local implementation merge. Firmware changes, capability enablement, physical acceptance, and remote push remain outside this resumption.
|
|
|
|
## Plan
|
|
|
|
### 2026-08-16 Cross-Platform Robot Hotspot Integration Plan
|
|
|
|
1. Merge reviewed source `c1326a2` into local `main` with a normal no-ff merge while preserving source history; exclude the source-owned task record and proposal from the integrated tree.
|
|
2. Accept the proposal as a new ADR that supersedes only ADR-002's manual OS hotspot-selection step, and reconcile current state, decision/architecture/domain/evidence/commitment memory without overclaiming platform or physical-device validation.
|
|
3. Run the 132-test Robot hotspot selection, full unit suite, typecheck, lint, production build, task-aware document gates, whitespace/unmerged-entry checks, and exact topology checks on the staged merge.
|
|
4. Obtain an independent read-only Sol PASS/FAIL integration review, fix any blocking findings, create the local no-ff merge commit, and leave firmware and remote push untouched.
|
|
|
|
### 2026-08-16 Guided Hotspot Default-On Integration Plan
|
|
|
|
1. Merge reviewed source `b78fc07` into local `main` with a normal no-ff merge, preserve feature history, and exclude the source-owned task record from the integrated tree.
|
|
2. Reconcile ADR-002, decision index, success criteria, Robot architecture/domain/current-state memory, glossary, and support wording from default-off/planned enablement to default-on with exact `=0` rollback; preserve all non-default security and evidence caveats.
|
|
3. Run Robot focused tests, typecheck, scoped lint, production build, bounded-concurrency full tests, task-aware document drift, whitespace/topology checks, and an independent final Sol integration review.
|
|
4. Commit the verified local merge. Do not modify firmware or push remotely.
|
|
|
|
### 2026-08-16 Robot Guided Hotspot Binding Implementation Integration Plan
|
|
|
|
1. Merge reviewed source commit `b7a1590` into local `main` with a normal no-ff merge, preserve the source parent, and exclude the source-owned task record from the final main tree.
|
|
2. Update only canonical statements that still say the accepted V1 is planned/not implemented; record `b7a1590` under `Integrated Through` while retaining default-off and release-gate wording.
|
|
3. Re-run the 90 focused tests, full unit suite, typecheck, scoped lint, production build, Electron Robot navigation smoke, project-document gates, and whitespace/topology checks on the merged tree.
|
|
4. Obtain an independent read-only Sol PASS/FAIL integration review, commit the verified no-ff merge, and leave firmware, capability enablement, physical acceptance, and remote push untouched.
|
|
|
|
### 2026-08-16 Robot Guided Hotspot Binding V1 Decision Integration Plan
|
|
|
|
1. Verify source commit `14afe4a`, its independent PASS reviews, the existing Robot implementation facts, and explicit human acceptance of the minimal-firmware V1.
|
|
2. Create an accepted ADR and promote the planned default-off capability, fixed portal action, Binding retry/restart semantics, release gates, and terminology into canonical project memory without claiming implementation is present.
|
|
3. Run project-document structure/drift and whitespace checks, then obtain an independent read-only Sol PASS/FAIL review.
|
|
4. Commit the canonical decision locally; start product implementation only from the accepted main baseline and do not modify firmware.
|
|
|
|
### 2026-08-16 Robot Configuration Catalog Integration Plan
|
|
|
|
1. Merge reviewed source commit `fe55dee` into local `main` with a normal no-ff merge while preserving source history.
|
|
2. Exclude the source-owned task record from the final `main` tree and promote the accepted safe catalog boundary into canonical architecture/current-state memory.
|
|
3. Re-run the Robot catalog/page/Main regression suite, typecheck, lint, production build, project-document gates, and whitespace checks.
|
|
4. Obtain an independent read-only Sol PASS/FAIL review; do not push remotely as part of this resumption.
|
|
|
|
1. Fetch `origin/main` and verify local `main`, the remote tip, and reviewed AI Canvas source commit topology.
|
|
2. Merge `22378ef` normally into local `main` without rebase/reset, and retain the source commit as a dedicated second parent.
|
|
3. Promote the bidirectional Gateway and Quote-reconciliation facts into canonical project memory.
|
|
4. Re-run focused/full checks, production build, project-document gates, and independent Sol review.
|
|
5. Commit the verified local merge; do not push unless the user separately requests it and authentication is available.
|
|
|
|
### 2026-08-15 Context Compaction Integration Plan
|
|
|
|
1. Form the already verified feature worktree into a dedicated source commit without changing its reviewed files.
|
|
2. Merge the direct child into local `main` with a normal no-ff merge and preserve source history.
|
|
3. Promote the per-session persistent compaction timeline and `session.compacted != session.idle` invariant into canonical current-state, architecture, evidence, and upgrade commitments.
|
|
4. Re-run focused/full unit checks, typecheck, lint, production build, Electron E2E, document drift, and whitespace checks on the merged tree.
|
|
5. Obtain an independent read-only Sol PASS/FAIL review, record the result, and keep remote push outside this request.
|
|
|
|
### 2026-08-15 Robot Configuration Loading Integration Plan
|
|
|
|
1. Merge reviewed source commit `1bcd519` into local `main` with a normal no-ff merge while preserving source history.
|
|
2. Exclude the source-owned task record from the final `main` tree so the integration task does not own or rewrite another task's project document.
|
|
3. Run the 7-file Robot/Main regression suite, typecheck, scoped lint, build, project-document gates, and whitespace checks.
|
|
4. Obtain an independent read-only Sol PASS/FAIL review, record exact topology and verification, and keep remote push outside this request.
|
|
|
|
### 2026-08-15 Robot Configuration Schema Integration Plan
|
|
|
|
1. Merge reviewed source commit `ea75b06` into local `main` with a normal no-ff merge while preserving source history.
|
|
2. Exclude the source-owned task record from the final `main` tree, leaving it reachable on the source commit and feature branch.
|
|
3. Re-run the AI hardware Main/Renderer/page regression selection, typecheck, scoped lint, build, project-document gates, and whitespace checks.
|
|
4. Obtain an independent read-only Sol PASS/FAIL review and keep the pre-existing remote-push blocker separate.
|
|
|
|
## Outcome
|
|
|
|
- On 2026-08-16, formed reviewed in-app Robot hotspot source commit `c1326a298026a697181c822cdd3062cc96c3aa2d` as the exact direct child of local `main` at `abecd5f34485ab467e5f032c618083d88e34b74d`, then started a normal `--no-ff --no-commit` merge. Git reported no textual conflicts.
|
|
- The staged main tree now owns Windows WLAN and macOS CoreWLAN/CoreLocation scan/connect/verify adapters behind one bounded Main Module and typed Host seam. Renderer exposes explicit candidate selection, safe recovery, system-Wi-Fi fallback, and the unchanged fixed-Portal/six-digit Binding continuation.
|
|
- Excluded the source-owned task record and proposal from the integrated tree while preserving both on source commit/branch `c1326a2`. Accepted ADR-003 and reconciled ADR-002, current state, architecture, domain rules, evidence, and release commitments without claiming signed macOS or physical-Robot acceptance.
|
|
- This resumption is a local `main` merge only. It changes no firmware and performs no remote push; exact `NIANCODE_AI_HARDWARE_GUIDED_HOTSPOT_BINDING=0` remains rollback.
|
|
- Created no-ff merge commit `61bd8bd6382e4300c81887f2d49216373d4d4477` with local baseline `abecd5f34485ab467e5f032c618083d88e34b74d` as first parent and reviewed source `c1326a298026a697181c822cdd3062cc96c3aa2d` as second parent.
|
|
- On 2026-08-16, started a normal `--no-ff --no-commit` merge of reviewed default-on source `b78fc07`; Git reported no textual conflicts. The feature task record remains reachable on the source commit/branch and is excluded from the integrated `main` tree.
|
|
- Integrated only the Main default change and focused route regressions: unset environment reports guided capability true; exact `NIANCODE_AI_HARDWARE_GUIDED_HOTSPOT_BINDING=0` and dependency-injected false report disabled. Renderer, Host wire shape, fixed portal, cloud Binding contract, credentials, and firmware are unchanged.
|
|
- Reconciled ADR-002, decision index, success criteria, current state, Robot architecture/domain/glossary, README, and commitments to default-on with exact `=0` rollback. The documents explicitly preserve unverified shipped-firmware, activation-issuer, native-opener, and physical-device evidence instead of claiming those checks passed.
|
|
- Created no-ff merge commit `027d36d7a91045e1dc434ad9ef7b25a40db4af22` with first parent `971865c256c2ddfe1f8ab4f8f4731a1f608599c0` and reviewed source second parent `b78fc07dba0d720e88c55667f8a36ebc50134f87`. Final independent integration re-review returned PASS. This local default-on resumption is complete; the long-lived integration task remains blocked only on its separate pre-existing authenticated remote-push scope, which this user request did not authorize.
|
|
- On 2026-08-16, started a normal `--no-ff --no-commit` merge of reviewed Guided Hotspot Binding source `b7a1590`; Git reported no textual conflicts. The source task record remains reachable on the source commit/feature branch and is excluded from the local `main` result.
|
|
- Integrated the default-off Main capability and strict local Host actions, fixed system-browser portal ownership, in-memory Renderer guided/direct Binding journey, safe conflict/retry/secret cleanup, and Bound-without-online semantics. No firmware file, BLE/Wi-Fi discovery, cloud claim route, or capability enablement was added.
|
|
- Reconciled canonical current state, system overview, and decision index from planned/not implemented to implemented/default off. Release gates remain exact firmware and issuer/validator verification, gate-on native-opener Electron coverage, and physical-device smoke.
|
|
- Final no-ff merge topology is prepared with first parent `54443232dd6a07dc1f3df5b33df00f665540497a` and reviewed source second parent `b7a1590ca132c971ffff177cb1e2aa47f96358cf`. This resumption is complete locally; the long-lived integration task remains blocked only on its separate pre-existing authenticated remote-push scope.
|
|
- Accepted source design commit `14afe4a` as ADR-002: V1 guides the user through the current firmware Hotspot portal and then reuses the existing six-digit Binding facade. The former Security 2/automatic-claim design remains deferred rather than silently mixed into V1.
|
|
- Promoted a narrow planned interface: Main-owned `guidedHotspotBinding` default false, fixed system-browser portal action for `http://192.168.4.1/`, Renderer-only guidance state, no Wi-Fi credential handling, and no claim that `bound` means online/ready.
|
|
- Kept implementation truth explicit: this integration step changes canonical documents only. Product behavior and firmware remain unchanged; pilot enablement is gated on exact shipped-image checks, activation-code contract checks, automated/Electron coverage, and physical-device smoke.
|
|
- On 2026-08-16, started a normal `--no-ff --no-commit` merge of reviewed Robot catalog/editor source `fe55dee`; Git reported no textual conflicts. The source task record remains reachable on the source commit and feature branch and is excluded from the local `main` result.
|
|
- Promoted the USER-scoped dynamic model/voice catalog boundary: Renderer receives only bounded display metadata through Electron Main and Works Square, all public catalog outcomes are no-store, and current unavailable values remain editable.
|
|
- Replaced avoidable configuration text/number inputs with catalog selects and bounded TTS sliders while preserving `clear_fields`, revision conflict, operation identity, and Main-owned credential boundaries.
|
|
- Created no-ff merge commit `c035273` with local baseline `7bef261` as first parent and reviewed source `fe55dee` as second parent.
|
|
- On 2026-08-15, merged reviewed Robot configuration-schema source `ea75b06` into local `main` without textual conflict and excluded its source-owned task record from the final tree.
|
|
- Created no-ff merge commit `9738e1c5aebad6237f436c8c94abaf877eb3d175` with local baseline `a26a53a7f4b1326be3084955029a65ef1f79929a` as first parent and reviewed source `ea75b0618a793928878edad736f0837ac9e4adc1` as second parent.
|
|
- Confirmed the failure was transport-contract drift, not a Xiaozhi/Works DTO failure: response compression weakened the origin strong revision ETag to canonical `W/\"0\"`; Main now accepts only canonical strong or weak numeric response tags whose revision equals the strictly projected body.
|
|
- Preserved optimistic concurrency and security boundaries: PATCH/PUT still send strong `If-Match`, and authentication, idempotency, size/deadline limits, fixed paths, and error redaction are unchanged.
|
|
- On 2026-08-15, formed the reviewed Robot configuration-loading fix as source commit `1bcd51964da04e5d80b461547d6bcccfafc0bec9` on its isolated feature branch.
|
|
- Fetched `origin/main` and verified both local and remote-tracking `main` were exactly `a4050f0a6567e8203630bf5d16b57f00c45caab8` with zero ahead/behind before integration.
|
|
- Started a normal `--no-ff --no-commit` merge of `1bcd519`; Git reported no textual conflicts. The source task record remains reachable on source commit `1bcd519` and its feature branch and is excluded from the `main` result to preserve project-document ownership boundaries.
|
|
- The merged code gives Robot configuration reads an explicit terminal error/retry state and keeps the same upstream deadline active through bounded response-body read and parse. Strict configuration DTO, ETag/revision, Main-owned authentication/idempotency, size limits, and error redaction remain unchanged.
|
|
- Independent final integration review returned `PASS` after validating merge topology, remote baseline, staged-tree ownership, behavior, tests, build, and project-document gates.
|
|
- Created no-ff merge commit `f6f7f21941bc5f443eb5e7800066f5b870313680` with local `main` baseline `a4050f0a6567e8203630bf5d16b57f00c45caab8` as first parent and source `1bcd51964da04e5d80b461547d6bcccfafc0bec9` as second parent.
|
|
- On 2026-08-13, resumed this integration task for the user's local-main merge request. `git fetch origin main --prune` succeeded and confirmed `origin/main=22add3f01f2b7cb6318495e8294db006f6f18abf`.
|
|
- Verified source commit `aba5cae286807093cf4ef643fe9f498050985c31` is a direct descendant of that remote tip and that local `main` is its ancestor, then fast-forwarded local `main` to `aba5cae` without rebase, reset, stash use, or conflict.
|
|
- The existing shared `stash@{0}` was not applied, popped, or dropped.
|
|
- Promoted the Robot/Main-owned hardware boundary into canonical current-state and system-overview documents. This request does not authorize or claim a remote push.
|
|
- Kept the feature task record on source commit `aba5cae` and removed it again from `main` in `22724c7`, so this integration task does not rewrite another task's owned project document; source evidence remains reachable on the feature branch.
|
|
- `git fetch origin main --prune` resolved the authoritative remote tip to
|
|
`f4113a872f7cd6aee6829c9597c882846bc4085b`. The histories had diverged at
|
|
`253bad8b40c8cd20a25362006f5d80a4e5c4cd4a`: local `main` contained 18
|
|
integration/preparation commits not on the remote, while the remote contained
|
|
one consolidation commit not on local `main`.
|
|
- Merged `origin/main` with `--no-ff --no-commit`, keeping local `main` as the
|
|
first parent. The only textual conflicts were `README.md` and
|
|
`.project-docs/20-architecture/data-flow.md`; both were reconciled
|
|
semantically instead of choosing either side wholesale.
|
|
- Accepted the remote product consolidation that removes the independent Device
|
|
Preview capability and the bundled Superpowers distribution, while retaining
|
|
the reviewed local Updater, image-to-image, first-chat, AI-proxy, publishing,
|
|
and `/deliverables` behavior. Canonical architecture/current-state documents
|
|
were updated to remove stale live Device Preview claims and record remote tip
|
|
`f4113a8` under `Integrated Through`.
|
|
- Preserved the remote startup warmup, curated course skills, per-Agent model
|
|
ownership, recursive Skill details, OpenCode path resolution, authentication
|
|
hardening, light visual consolidation, fonts, and window material changes.
|
|
- Corrected an upstream course-skill manifest mismatch by retiring the absent
|
|
`deploy-publish-check` skill instead of advertising it as bundled, with a
|
|
regression assertion in the manager suite.
|
|
- Corrected merge-exposed test defects without weakening production behavior:
|
|
project-config request mocking now receives `RequestInit`; the Windows runtime
|
|
queue test injects a deterministic port-owner lookup; the legacy agent-folder
|
|
test accepts the intentionally absent directory; and an unused test binding
|
|
was removed.
|
|
- Rebased the Electron E2E contract on the consolidated UI: Code/Canvas enter
|
|
through the module chooser, hidden module actions are expanded before use,
|
|
Models is reached through the account/settings flow, Provider seeding uses the
|
|
Main-owned Host API, and project-backed chat mocks return valid initialized
|
|
configuration. Two Channels specs were removed because the remote product
|
|
consolidation removed their UI, route, and Host API rather than relocating
|
|
them. Stable test IDs were restored to the retained proxy settings controls.
|
|
- Created merge commit
|
|
`29c458f07dd40bf8f66d4fbb2acb6fe2a1277cd7` with local preparation commit
|
|
`1c85bc04547a72ec76074480245f623e37f9cd30` as first parent and fetched remote
|
|
tip `f4113a872f7cd6aee6829c9597c882846bc4085b` as second parent.
|
|
- Independent read-only Sol review returned `PASS` with no P0-P3 findings after
|
|
checking topology, retained local features, remote consolidation, course
|
|
skill parity, documentation, E2E strength, security boundaries, and
|
|
dependency consistency.
|
|
- A normal `git push origin main` was attempted and rejected by the remote with
|
|
`Failed to authenticate user`. A read-only batch SSH check also returned
|
|
`Permission denied (publickey,...)`; the configured HTTPS credential helper
|
|
has no usable username/secret for this host. No force option was used and no
|
|
remote ref changed. Local `main` and the completed merge remain intact.
|
|
- Completion is blocked only on the user authenticating this machine for
|
|
`git.nianxx.cn`. Keep this integration task owned and do not release it until
|
|
a normal push succeeds and `origin/main` is verified equal to local `HEAD`.
|
|
- On 2026-08-14, the user requested a local-main merge of reviewed AI Canvas source `22378ef`. A fresh fetch verified local and remote `main` were both `88f9ee8`, and the source was exactly one descendant commit with no unrelated overlap.
|
|
- Started a normal `--no-ff --no-commit` merge of `22378ef`; Git reported no textual conflict. The source task record remains intact on source commit `22378ef` and its feature branch, and is excluded from the `main` result to preserve task-document ownership boundaries.
|
|
- Promoted the accepted source facts into canonical current-state, architecture, data-flow, business-rule, evidence, and commitment records: connected Conversation commands/events are bidirectional over WebSocket; REST fallback is transport-only and idempotent; Quote task recovery remains Workspace-owned while Conversation writes retain generation guards.
|
|
- Merge validation exposed a pre-existing AI Hardware test race: the edit button is rendered disabled while configuration loads, but four tests clicked it after waiting only for existence. Under full-suite load the browser correctly ignored the disabled click. The tests now wait for the button to become enabled and for the dialog heading; production Robot behavior is unchanged.
|
|
- The first independent AI Canvas merge review returned `FAIL` on two integration gaps and one safety subfinding: stale confirmation task-refresh errors could overwrite a newer Conversation error, top-level structured WebSocket command errors lacked focused coverage, and unknown Gateway messages could expose upstream details. Confirmation reconciliation now uses an internal task refresh without UI-error side effects; explicit user refreshes retain their error behavior. Matching WebSocket errors are tested as non-retryable, and unknown codes project a fixed Chinese fallback instead of the server message.
|
|
- The second independent Sol review returned `PASS` with no blocking findings after those corrections. The verified no-ff merge is ready to commit with `88f9ee8` as first parent and `22378ef` as second parent.
|
|
- This resumption intentionally does not push. The previous remote authentication follow-up remains unchanged and does not block completion of the user's requested local merge.
|
|
- On 2026-08-15, formed the reviewed context-compaction implementation into source commit `fd9b5b46a913c515e94e4e26f185d43866c2581f`; its parent is exactly current local `main` (`953b0f4`) and the source worktree is clean.
|
|
- Started a normal `--no-ff --no-commit` merge of `fd9b5b4` into local `main`; Git reported no textual conflicts. The source task record remains unchanged on `fd9b5b4` and its feature branch and is excluded from the `main` result to preserve task-document ownership boundaries.
|
|
- Promoted the accepted compaction facts into canonical current-state, module map, data flow, evidence, and upgrade commitments: compaction is a persistent per-session transcript event, completed status is monotonic, and `session.compacted` cannot end the run or release queued prompts.
|
|
- Created no-ff merge commit `1d0878bb5b18e9b17b147a4d43ea0861913da785` with `953b0f4` as first parent and source `fd9b5b4` as second parent.
|
|
- The first independent integration review returned `FAIL` on two uncovered edges: HTTP polling-only idle could release the run while leaving its compaction running, and cold busy hydration could misclassify a historical compaction Part as current.
|
|
- Closed both findings in `7a811590c4943b7b1b7ea5f3b4d3ce3ce05622a5`: compact polling idle now completes only the matching `runID + generation` event before queue release, and cold hydration keeps a native Part running only when current transcript state provides a matching running identity. Historical compactions remain completed during a later ordinary busy run.
|
|
- On 2026-08-16, staged a local no-ff merge of reviewed Robot device-selection source `87a95b4` onto `main` at `3b46697`; Git reported no textual conflicts. The source task record remains intact on the source commit and feature branch and is excluded from the `main` result to preserve task-document ownership boundaries.
|
|
- The Robot workspace now derives device rows and per-agent counts from the selected agent, keeps activation-code binding fixed to that current agent, and retains the editable agent selector only for explicit reassignment. This is a Makelore-only correction and does not add device discovery, manual MAC entry, or new Works Square/Xiaozhi endpoints.
|
|
- This resumption intentionally performs a local merge only. It does not fetch or push, and the existing remote-authentication follow-up for this long-running integration task remains unchanged.
|
|
- Created no-ff merge commit `4ff1a7a8461c01de4d1630a3db55b0942fc37ec7` with `3b4669722e0ea671909beb039da1e07c2483341a` as first parent and reviewed source `87a95b4cecf20fce7912305d2e6e2c063ddcbae4` as second parent.
|
|
|
|
## Verification
|
|
|
|
- 2026-08-16 cross-platform Robot hotspot source final Sol review — `PASS` on Standards and Spec with no P0-P3 findings; macOS termination/cancellation races, Renderer post-connect state, default-on rollback, security boundaries, and firmware-zero-change were confirmed.
|
|
- Staged merged-main Robot hotspot selection — 4 files / 132 tests passed.
|
|
- Staged merged-main `pnpm test` — 157 files / 1796 tests passed.
|
|
- Staged merged-main `pnpm run typecheck` and `pnpm run lint:check` — passed; lint retained 0 errors and the same 6 warnings outside the merge paths.
|
|
- Staged merged-main `pnpm run build:vite` — Renderer, Electron Main, and Preload passed; emitted Windows and macOS Main chunks, with only existing mixed-import/chunk-size warnings.
|
|
- Staged merged-main Electron module-navigation E2E — 2/2 passed after rebuilding the production Renderer/Main/Preload tree. The fixture does not simulate a physical Robot or signed macOS association and is not claimed as native acceptance.
|
|
- Integration task-aware document drift, project-document structure, staged/unstaged whitespace, source-record exclusion, and unmerged-entry checks passed before independent integration review.
|
|
- Independent final cross-platform Robot hotspot integration review — `PASS` on Standards and Spec with no P0-P3 findings. It confirmed source-code identity, exact merge topology, source-record exclusion, Main-owned native/security boundaries, ADR-003's narrow supersession, canonical consistency, and honest residual release gates.
|
|
- Post-commit topology confirmed exact parents `abecd5f` and `c1326a2`, source ancestry on `main`, branch `main`, and a clean product worktree before this evidence-only task-record update.
|
|
- 2026-08-16 default-on feature final Sol review — `PASS`, no P0-P3 findings; independently confirmed default-on, exact environment opt-out, dependency-injection opt-out, fixed portal ownership, local-before-Works-token behavior, error redaction, unchanged Binding contract, and zero firmware changes.
|
|
- Merged-main Robot selection — 3 files / 94 tests passed, including fresh-module exact `=0` rollback coverage.
|
|
- Merged-main `pnpm run typecheck` and scoped ESLint on the two source files — passed.
|
|
- Merged-main `pnpm run build:vite` — Renderer, Electron Main, and Preload passed; only existing mixed-import/chunk-size warnings remain.
|
|
- Merged-main bounded full suite `pnpm vitest run --maxWorkers=4` — 156 files / 1758 tests passed. The source task separately records why bounded concurrency was used after the unrelated OpenCode port-release test hit its 10-second limit under default all-suite parallelism but passed 40/40 in isolation.
|
|
- Merged-main Electron module-navigation smoke — 1/1 passed through the existing packaged Main/Renderer fixture and entered `/ai-hardware`. The fixture still cannot authenticate a Robot overview or observe the native external opener, so that deeper E2E remains a tracked release-validation gap.
|
|
- `check_project_docs.py` — passed; all required planning/canonical files exist.
|
|
- Integration task-aware document drift, task registry doctor, staged/unstaged whitespace checks, unmerged-entry check, and exact merge-base/parent checks — passed.
|
|
- Post-commit topology check confirmed `027d36d` has the exact two reviewed parents, `b78fc07` is a `main` ancestor, and the product worktree was clean before this evidence-only task-record update.
|
|
- Independent final default-on integration review initially returned `FAIL`: default-on was incorrectly expanded into explicit SoftAP/plain-HTTP risk acceptance, one obsolete follow-up still said “before pilot enablement,” and the default-on test depended on ambient environment state. Canonical wording now records only the explicit default-on/current-firmware decision while retaining the security risk; the follow-up targets the next default-on release/compatibility claim; the test deletes the environment variable and reloads the route module before asserting true. Final re-review returned `PASS` with no P0-P3 findings and independently passed route tests both normally and with ambient `NIANCODE_AI_HARDWARE_GUIDED_HOTSPOT_BINDING=0`.
|
|
- 2026-08-16 Robot device-selection merged-tree regression selection — 3 files / 92 tests passed.
|
|
- Robot device-selection merged-tree `pnpm run typecheck` and scoped ESLint — passed.
|
|
- Staged/unstaged whitespace checks and unmerged-entry checks — passed before independent review.
|
|
- Independent Robot device-selection integration review — `PASS`, no P0-P3 findings; it confirmed selected-agent device isolation and counts, immutable bind target, retained reassignment semantics, source-record exclusion, and the Makelore-only API boundary.
|
|
- Post-commit topology verification confirmed both expected parents, the source commit as a `main` ancestor, a clean product worktree before this evidence-only update, and task-aware document drift passing.
|
|
- 2026-08-16 Guided Hotspot source final Sol re-review — `PASS`, no P0-P3 findings after fixed-address copy recovery, conflict/already-bound overview refresh, and complete state-machine/remount coverage were added.
|
|
- Staged local-main merge Robot selection — 3 files / 91 tests passed.
|
|
- Staged local-main merge `pnpm test` — 156 files / 1755 tests passed.
|
|
- Staged local-main merge `pnpm run typecheck`, scoped ESLint, and `pnpm run build:vite` — passed; the production build retains only existing chunk-size and mixed-import warnings.
|
|
- Staged local-main merge Electron module-navigation smoke — 2/2 passed, including entry into the Robot route. Gate-on native external-open observation remains an explicit release prerequisite rather than a claimed test.
|
|
- Independent staged-merge review initially returned `FAIL` with one P2 and two P3 findings: ADR implementation status remained planned, current-state's update date was stale, and Escape/overlay dismissal during a pending native opener could let an old promise affect a reopened wizard. The documents now state implemented/default-off with the current date; portal-opening sessions reject all close attempts, with an Escape/remount regression test. Focused/full/typecheck/lint/build checks were rerun before re-review.
|
|
- Independent staged-merge re-review — `PASS`, no P0-P3 findings. It confirmed all three findings closed, source-task-record exclusion, exact merge parents, default-off/release-gate truth, and no old opener promise can affect a reopened guided session.
|
|
- 2026-08-16 ADR-002 integration `check_project_docs.py`, task-aware `check_doc_drift.py`, and `git diff --check` — passed in the Integration owner worktree.
|
|
- Independent ADR-002 canonical integration review initially returned `FAIL` on two P2 documentation inaccuracies: a shortened Host wire envelope and an overclaim about deployed firmware. Both were corrected; final re-review returned `PASS` with no P0-P3 findings.
|
|
- The final review confirmed only canonical documents and this integration task record changed; no product source, source-task record, or firmware file was modified in this acceptance step.
|
|
- 2026-08-16 source and staged-merge Robot catalog regression selection — 5 files / 84 tests passed.
|
|
- 2026-08-16 Works Square catalog adapter/API suite — 131 tests passed; one existing Starlette/httpx deprecation warning.
|
|
- 2026-08-16 Makelore typecheck, focused ESLint, and production `build:vite` — passed; only existing chunk-size and mixed-import warnings remain.
|
|
- 2026-08-16 independent source review — `PASS`, with no P0-P3 findings after all-outcome no-store and 40px target fixes.
|
|
- 2026-08-16 independent final staged-merge review — `PASS`, no P0-P3 findings; topology, net diff, source-record exclusion, canonical promotion, and merged security/UI behavior were confirmed.
|
|
- Robot configuration-schema source TDD recorded `2 failed, 23 passed` before the fix with `AI_HARDWARE_INVALID_ETAG` for the deployed weak-tag shape.
|
|
- Merged-tree AI hardware route suite passed `25/25`; the seven-file Robot/Main regression selection passed `78/78`.
|
|
- Merged-tree `pnpm run typecheck`, scoped ESLint, and `pnpm run build:vite` passed; build retains only existing chunk-size and mixed-import warnings.
|
|
- Merged-tree `pnpm test` passed `156` files / `1727` tests.
|
|
- Task-aware project-document drift, task registry doctor, staged whitespace checks, and unmerged-entry checks passed.
|
|
- Independent final Robot response-ETag integration review returned `PASS` with no P0-P2 findings; it confirmed weak ETags are accepted only on responses, write preconditions remain strong, and the parsed ETag must equal the projected DTO revision.
|
|
- 2026-08-15 Robot configuration-loading merged-tree regression selection — 7 files / 74 tests passed.
|
|
- Robot merged-tree `pnpm run typecheck` — passed.
|
|
- ESLint on the four merged TypeScript/TSX files — passed.
|
|
- `pnpm run build:vite` — Renderer, Electron Main, and Preload production builds passed; only the existing chunk-size and mixed-import warnings remain.
|
|
- Staged/unstaged whitespace checks and task-aware project-document drift — passed before final review.
|
|
- Independent Robot configuration-loading integration review — `PASS`, no blocking findings.
|
|
- Final merge topology — first parent `a4050f0`, second parent `1bcd519`, merge commit `f6f7f21`.
|
|
- Robot source closeout before integration: 8 focused files / 72 tests passed; TypeScript and task-aware document drift passed.
|
|
- Post-fast-forward `pnpm test`: 156 files / 1681 tests passed.
|
|
- `pnpm run typecheck` and `pnpm run build:vite`: passed; build retains only existing chunk/dynamic-import warnings.
|
|
- Real loopback Makelore Main-to-Works Square contract: 8/8 passed.
|
|
- `pnpm run lint:check`: 0 errors and 6 existing warnings.
|
|
- Robot module-navigation Electron E2E: 2/2 passed.
|
|
- `git diff --check`, task-aware document drift, and task registry doctor passed on clean local `main`; independent final review returned PASS for the local Robot integration. The task remains blocked only on its pre-existing remote-push scope because Git authentication is unavailable.
|
|
- `pnpm install --frozen-lockfile` — passed.
|
|
- High-risk focused Vitest selection — 19 files, 494/494 passed.
|
|
- `pnpm test` — 146 files, 1574/1574 passed.
|
|
- `pnpm run typecheck` — passed.
|
|
- `pnpm run lint:check` — passed with 0 errors and 6 existing warnings.
|
|
- `pnpm run build:vite` — passed.
|
|
- `pnpm run test:electron:windows` — 3/3 passed.
|
|
- `pnpm run test:e2e` — rebuilt Renderer/Main/Preload and passed 24/24.
|
|
- `git diff --check` and `git diff --cached --check` — passed.
|
|
- Independent final review — `PASS`, no P0-P3 findings.
|
|
- Merge topology — verified first parent `1c85bc0`, second parent `f4113a8`.
|
|
- Push — attempted normally, rejected before ref update because remote
|
|
authentication is unavailable on this machine.
|
|
- 2026-08-14 AI Canvas focused selection — 3 files / 74 tests passed on the merged `main` tree.
|
|
- AI Hardware readiness regression — full `ai-hardware-page.test.tsx` passed 24/24 after replacing existence-only clicks with enabled/dialog readiness waits.
|
|
- `pnpm test` — the original full-suite loop failed twice at the same disabled-button race before the test hardening; the post-fix run passed 156 files / 1687 tests.
|
|
- `pnpm run typecheck` — passed after the integration test hardening.
|
|
- `pnpm run lint:check` — passed with 0 errors and 6 unchanged warnings outside the merge paths.
|
|
- `pnpm run build:vite` — Renderer, Electron Main, and Preload production builds passed; existing chunk-size and mixed-import warnings remain.
|
|
- Post-review focused checks — Works Square adapter 27/27 and Image Workspace Store coverage passed, including top-level WebSocket errors, unknown-message redaction, A→B task-refresh failure isolation, and explicit-refresh error reporting.
|
|
- Final post-review `pnpm test` — 156 files / 1691 tests passed.
|
|
- Final post-review `pnpm run typecheck` — passed.
|
|
- Final post-review `pnpm run lint:check` — passed with 0 errors and the same 6 warnings outside the merge paths.
|
|
- Final post-review `pnpm run build:vite` — Renderer, Electron Main, and Preload passed; only the existing mixed-import and chunk-size warnings remain.
|
|
- Second independent final review — `PASS`; it re-ran 4 files / 102 tests plus typecheck, found no unmerged entries or unstaged changes, and confirmed the previous state-isolation, structured-error coverage, and redaction findings are closed.
|
|
- Final project-document ownership drift, structure checks, and staged/unstaged whitespace checks — passed before the merge commit.
|
|
- 2026-08-15 context-compaction focused selection — 3 files / 236 tests passed on the merged `main` tree.
|
|
- 2026-08-15 `pnpm test` — 156 files / 1712 tests passed on the merged `main` tree.
|
|
- 2026-08-15 `pnpm run typecheck` — passed.
|
|
- 2026-08-15 `pnpm run lint:check` — passed with 0 errors and the same 6 existing warnings outside the merge paths.
|
|
- 2026-08-15 `pnpm run build:vite` — Renderer, Electron Main, and Preload production builds passed; only existing chunk-size and mixed-import warnings remain.
|
|
- 2026-08-15 Electron E2E selection — `tests/e2e/opencode-slash-commands.spec.ts` passed 3/3, including the manual compaction running-to-completed timeline transition.
|
|
- 2026-08-15 staged/unstaged whitespace checks and project-document structure passed; task-aware drift passed after retaining the source task record only on the feature branch.
|
|
- Post-review focused selection — 3 files / 239 tests passed, including polling-only idle completion, queued prompt release, cold historical hydration, realtime native Part preservation, and completed-state monotonicity.
|
|
- Post-review `pnpm test` — 156 files / 1715 tests passed.
|
|
- Post-review `pnpm run typecheck` and `pnpm run lint:check` — passed; lint retained only the same 6 warnings outside changed files.
|
|
- Post-review `pnpm run build:vite` — Renderer, Electron Main, and Preload passed with only existing warnings.
|
|
- Post-review Electron E2E selection — 3/3 passed.
|
|
- Final context-compaction integration re-review — `PASS`; no remaining P0-P3 findings. The reviewer confirmed polling-idle completion precedes queue release, cold hydration requires a matching current running identity, `main` is clean, and merge/fix/documentation topology is correct.
|
|
|
|
## Follow-ups
|
|
|
|
- Before claiming complete compatibility for the default-on Robot journey, verify the exact shipped firmware/fixed portal, six-digit issuer/validator freshness and consumption semantics, real Host API/native opener behavior, and a physical-device provisioning + Binding smoke. Keep exact `NIANCODE_AI_HARDWARE_GUIDED_HOTSPOT_BINDING=0` in support rollback instructions.
|
|
- The earlier default-false production instruction is superseded by the user's explicit default-on decision. Keep exact value `0` only as rollback, and do not touch `D:\Datas\HardwareProjects\xiaozhi-esp32-firmware` for this V1.
|
|
- Before the next default-on release or any complete-compatibility claim, identify the exact shipped Robot component/firmware image, verify that the deployed issuer produces six ASCII digits with compatible freshness/consumption semantics, and pass a real device smoke through the Host API/Electron flow.
|
|
- Deploy matching Xiaozhi and Works Square catalog endpoints before releasing this client; otherwise the editor preserves current values but cannot populate dynamic choices.
|
|
- The Robot integration is complete on local `main`. Production still requires matching Works Square/Xiaozhi deployment, feature configuration, credentials, and a real one-time activation-code smoke.
|
|
- Stable mutation operation IDs are retained across ambiguous retries in the running app but are not persisted across an application restart.
|
|
- Signed packaged updater smoke and live Works Square/Bailian image-to-image
|
|
smoke remain external release gates; this Git synchronization does not claim
|
|
that either production environment has been exercised.
|
|
- A cancelled-before-spawn OpenCode restart can still spend roughly one second
|
|
probing a Windows port owner. The queue regression is now deterministic; a
|
|
future performance change should distinguish an attached runtime before
|
|
skipping that lookup so attached-runtime restart behavior remains intact.
|
|
- Authenticate `git.nianxx.cn` through the local Git credential manager or an
|
|
authorized SSH key, then resume this same task to push and verify remote tip
|
|
equality. Do not paste a personal access token into the task conversation.
|
|
- When bundled OpenCode or the model context profile changes, run a real long-context compaction smoke to reconfirm native Part fields and event ordering; the current Electron E2E uses a controlled EventSource through the real Store/Renderer lifecycle rather than launching OpenCode.
|
|
|
|
## Promotion Candidates
|
|
|
|
- The default-on candidate from `b78fc07` was promoted into ADR-002, current state, decision/success criteria, Robot architecture/domain/glossary, README, and a concrete release-validation commitment. No unresolved canonical candidate remains; the missing native/physical evidence is tracked as a pending commitment rather than overclaimed.
|
|
- The implementation truth from `b7a1590` was originally promoted as implemented/default-off. `b78fc07` now supersedes only that default; its former enablement evidence requirements remain tracked as default-on release validation and rollback commitments.
|
|
- The Guided Hotspot Binding V1 candidate from `14afe4a` was promoted into ADR-002, success criteria, system/module/data-flow architecture, business rules, glossary, and current state. No unresolved candidate remains for this design acceptance.
|
|
- The context-compaction source candidate was promoted into current state, module map, data flow, evidence, and upgrade commitments. No unresolved candidate remains for this local merge.
|