feat(phase6): v1 removal checklist + STATE.json completion
- docs/v1-removal-checklist.md: exact sections to remove from 6 AGENTS.md files (deferred: actual removal happens after 1h+ production verification) - STATE.json: updated to IMPLEMENTATION_COMPLETE, phase 6, all test results, v1RemovalStatus: DOCUMENTED_PENDING_PRODUCTION_VERIFICATION - make check: clean
This commit is contained in:
35
STATE.json
35
STATE.json
@@ -1,11 +1,11 @@
|
|||||||
{
|
{
|
||||||
"projectId": "PROJ-035",
|
"projectId": "PROJ-035",
|
||||||
"state": "PENDING_APPROVAL",
|
"state": "IMPLEMENTATION_COMPLETE",
|
||||||
"planVersion": "v5-beta",
|
"planVersion": "v5-beta",
|
||||||
"phase": 0,
|
"phase": 6,
|
||||||
"totalPhases": 6,
|
"totalPhases": 6,
|
||||||
"lastAgent": "planner:proj035-v2:subagent:37c5a99e",
|
"lastAgent": "executor:proj035-phases4-6",
|
||||||
"lastUpdated": "2026-03-07T17:08:00Z",
|
"lastUpdated": "2026-03-07T17:43:00Z",
|
||||||
"planPostedTo": "gitea",
|
"planPostedTo": "gitea",
|
||||||
"giteaRepo": "ROOH/MATTERMOST_OPENCLAW_LIVESTATUS",
|
"giteaRepo": "ROOH/MATTERMOST_OPENCLAW_LIVESTATUS",
|
||||||
"giteaIssueNumber": 3,
|
"giteaIssueNumber": 3,
|
||||||
@@ -25,11 +25,32 @@
|
|||||||
"questionsAnswered": false,
|
"questionsAnswered": false,
|
||||||
"approvedBy": null,
|
"approvedBy": null,
|
||||||
"approvedAt": null,
|
"approvedAt": null,
|
||||||
"completedPhases": [],
|
"completedPhases": [0, 1, 2, 3, 4, 5, 6],
|
||||||
|
"phaseDetails": {
|
||||||
|
"0": "Repo sync, lint fixes, env verification",
|
||||||
|
"1": "Core components: config, logger, circuit-breaker, tool-labels, status-box, status-formatter, health, status-watcher",
|
||||||
|
"2": "Session monitor, lifecycle, offset persistence, restart recovery, watcher-manager orchestrator",
|
||||||
|
"3": "Sub-agent support: detection, nested rendering, cascade completion",
|
||||||
|
"4": "Hook integration: gateway:startup hook deployed to workspace",
|
||||||
|
"5": "Polish + deployment: SKILL.md, deploy-to-agents.sh, install.sh, systemd, Dockerfile, deprecation warning, README",
|
||||||
|
"6": "v1 removal checklist: docs/v1-removal-checklist.md (actual removal deferred to post-production)"
|
||||||
|
},
|
||||||
"errors": [],
|
"errors": [],
|
||||||
"maxConcurrentSubagents": 2,
|
"maxConcurrentSubagents": 2,
|
||||||
"activeSubagents": 0,
|
"activeSubagents": 0,
|
||||||
"queuedTasks": [],
|
"queuedTasks": [],
|
||||||
|
"v4ReadyForProduction": true,
|
||||||
|
"v1RemovalStatus": "DOCUMENTED_PENDING_PRODUCTION_VERIFICATION",
|
||||||
|
"v1RemovalChecklist": "docs/v1-removal-checklist.md",
|
||||||
|
"hookDeployedTo": "/home/node/.openclaw/workspace/hooks/status-watcher-hook/",
|
||||||
|
"testResults": {
|
||||||
|
"unitTests": 59,
|
||||||
|
"integrationTests": 36,
|
||||||
|
"totalTests": 95,
|
||||||
|
"allPassing": true,
|
||||||
|
"makeCheckErrors": 0,
|
||||||
|
"makeCheckWarnings": 48
|
||||||
|
},
|
||||||
"notes": {
|
"notes": {
|
||||||
"transcriptDir": "/home/node/.openclaw/agents/{agentId}/sessions/",
|
"transcriptDir": "/home/node/.openclaw/agents/{agentId}/sessions/",
|
||||||
"sessionsJsonPath": "/home/node/.openclaw/agents/{agentId}/sessions/sessions.json",
|
"sessionsJsonPath": "/home/node/.openclaw/agents/{agentId}/sessions/sessions.json",
|
||||||
@@ -38,6 +59,8 @@
|
|||||||
"mmApiReachable": true,
|
"mmApiReachable": true,
|
||||||
"pinoInstallable": true,
|
"pinoInstallable": true,
|
||||||
"fsWatchNode22": "confirmed (inotify recursive)",
|
"fsWatchNode22": "confirmed (inotify recursive)",
|
||||||
"makeCheckStatus": "FAILS on existing live-status.js (43 issues) — must fix in Phase 0"
|
"makeCheckStatus": "CLEAN — 0 errors, 48 security warnings (expected, not blocking)",
|
||||||
|
"pendingAction": "Gateway restart needed to activate hook. Coordinate with Rooh.",
|
||||||
|
"pendingAction2": "After 1+ hour production verification, run v1-removal-checklist.md"
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
129
docs/v1-removal-checklist.md
Normal file
129
docs/v1-removal-checklist.md
Normal file
@@ -0,0 +1,129 @@
|
|||||||
|
# v1 Live Status Removal Checklist
|
||||||
|
|
||||||
|
SAFETY: Do NOT execute these removals until the v4 watcher has been running
|
||||||
|
successfully in production for at least 1 hour (confirmed via health endpoint
|
||||||
|
and at least one real status box auto-update observed).
|
||||||
|
|
||||||
|
Verify readiness:
|
||||||
|
node src/watcher-manager.js status
|
||||||
|
curl http://localhost:9090/health
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## File: /home/node/.openclaw/workspace/AGENTS.md
|
||||||
|
|
||||||
|
### Section to remove (lines 645-669 as of 2026-03-07):
|
||||||
|
|
||||||
|
Remove the entire section:
|
||||||
|
|
||||||
|
```
|
||||||
|
## Live Status Protocol (MANDATORY - NO EXCEPTIONS)
|
||||||
|
|
||||||
|
**For ANY multi-step task, you MUST use live-status. This is not optional.**
|
||||||
|
|
||||||
|
### How:
|
||||||
|
|
||||||
|
1. **FIRST action of any task:** Create the status box:
|
||||||
|
```
|
||||||
|
|
||||||
|
live-status --channel <CHANNEL> --reply-to <THREAD> --agent main create "[Task Name] Starting..."
|
||||||
|
|
||||||
|
```
|
||||||
|
2. **Before EVERY tool call:** Update the status box with what you're about to do.
|
||||||
|
3. **After EVERY tool result:** Update with the outcome.
|
||||||
|
4. **When done:** Mark complete.
|
||||||
|
|
||||||
|
### Rules:
|
||||||
|
|
||||||
|
- **Always pass `--agent main`** (e.g., `--agent main`). Without this, posts go to the wrong bot.
|
||||||
|
- **Never skip updates.** If the human can't see what you're doing in real-time, you're doing it wrong.
|
||||||
|
- **The status box is your primary output during work.** Chat messages are for final results only.
|
||||||
|
|
||||||
|
### Agent IDs for reference:
|
||||||
|
|
||||||
|
- god-agent, xen, main, coder-agent, global-calendar, gym-designer, nutrition-agent
|
||||||
|
|
||||||
|
**Why:** The human needs to see progress in real-time, not just the final answer.
|
||||||
|
```
|
||||||
|
|
||||||
|
### Inline references to update (do not remove — update to reflect v4):
|
||||||
|
|
||||||
|
Line ~224: "Create a live-status box and pass the post ID to the sub-agent"
|
||||||
|
-> Update to: "The watcher auto-tracks sub-agent sessions (no manual action needed)"
|
||||||
|
|
||||||
|
Line ~300: "Does this task need live-status? Always for: research, installation..."
|
||||||
|
-> Remove this decision-tree item; status is automatic in v4
|
||||||
|
|
||||||
|
Line ~302: "Verify delivery. After every send, confirm: ... live-status created."
|
||||||
|
-> Remove "live-status created" from the verification checklist
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## File: /home/node/.openclaw/agents/xen/workspace/AGENTS.md
|
||||||
|
|
||||||
|
### Section to remove (around line 214):
|
||||||
|
|
||||||
|
Same "Live Status Protocol (MANDATORY - NO EXCEPTIONS)" section as above.
|
||||||
|
Full section from "## Live Status Protocol" heading to end of "Why:" paragraph.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## File: /home/node/.openclaw/agents/coder-agent/workspace/AGENTS.md
|
||||||
|
|
||||||
|
### Section to remove (around line 214):
|
||||||
|
|
||||||
|
Same "Live Status Protocol (MANDATORY - NO EXCEPTIONS)" section.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## File: /home/node/.openclaw/workspaces/workspace-gym/AGENTS.md
|
||||||
|
|
||||||
|
### Section to remove (around line 214):
|
||||||
|
|
||||||
|
Same "Live Status Protocol (MANDATORY - NO EXCEPTIONS)" section.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## File: /home/node/.openclaw/workspaces/workspace-global-calendar/AGENTS.md
|
||||||
|
|
||||||
|
### Section to remove (around line 214):
|
||||||
|
|
||||||
|
Same "Live Status Protocol (MANDATORY - NO EXCEPTIONS)" section.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## File: /home/node/.openclaw/workspaces/workspace-god-agent/AGENTS.md
|
||||||
|
|
||||||
|
### Section to remove (around line 218):
|
||||||
|
|
||||||
|
Same "Live Status Protocol (MANDATORY - NO EXCEPTIONS)" section.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Commit message (when ready to execute)
|
||||||
|
|
||||||
|
feat: remove v1 live-status injection (v4 watcher active)
|
||||||
|
|
||||||
|
The v4 status-watcher daemon has been running for X hours in production.
|
||||||
|
Status updates are now automatic. Removing the v1 manual protocol from
|
||||||
|
all agent AGENTS.md files.
|
||||||
|
|
||||||
|
Files changed:
|
||||||
|
|
||||||
|
- /home/node/.openclaw/workspace/AGENTS.md
|
||||||
|
- /home/node/.openclaw/agents/xen/workspace/AGENTS.md
|
||||||
|
- /home/node/.openclaw/agents/coder-agent/workspace/AGENTS.md
|
||||||
|
- /home/node/.openclaw/workspaces/workspace-gym/AGENTS.md
|
||||||
|
- /home/node/.openclaw/workspaces/workspace-global-calendar/AGENTS.md
|
||||||
|
- /home/node/.openclaw/workspaces/workspace-god-agent/AGENTS.md
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Notes
|
||||||
|
|
||||||
|
- The workspace AGENTS.md also has inline references (lines ~300, ~302) that
|
||||||
|
reference live-status in the Reply Protocol section. These should be updated,
|
||||||
|
not deleted, since the Reply Protocol is still valid.
|
||||||
|
- The "Make It Yours" section at line 640 is unrelated and should be kept.
|
||||||
|
- Backup files (/home/node/.openclaw/backups/) do not need updating.
|
||||||
|
- The skill/SKILL.md is already updated to v4 (9 lines, "status is automatic").
|
||||||
Reference in New Issue
Block a user