Skip to content

fix(skills): plan-collab undo reports undone and reviewed - #82

Merged
calvin-archastro merged 1 commit into
mainfrom
fix/plan-collab-undo-status
Oct 22, 2026
Merged

calvin-archastro merged 1 commit into
mainfrom
fix/plan-collab-undo-status

Conversation

@calvin-archastro

Copy link
Copy Markdown
Contributor

Problem

In a local end-to-end run (2026-10-21), after run.undo an agent following this skill sent run.status without undone/reviewed. The page kept the run marked "not reviewed" and blocked approval.

Change

Section 5 now says to send run.status with "undone":true and "reviewed":true, with an example. Sending both works with pages before and after https://archdev.ai/ArchAstro/firstlanding/pull/19524, which also makes the page count undone as reviewed. Plugin copies were regenerated with tools/plugin-gen.

Testing

  • npm run check: generated files match the catalog.
  • Live: on the local stack, the page cleared the run after run.status with these fields.

🤖 Generated with Claude Code

A local end-to-end run showed the page keeps an undone run unreviewed,
and approval blocked, unless run.status carries `undone: true` and
`reviewed: true`. The skill only said "send run.status". firstlanding
PR 19524 also makes the page treat `undone` as reviewed; sending both
works with pages before and after it.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@calvin-archastro
calvin-archastro merged commit 143ecf9 into main Oct 22, 2026
20 checks passed
@calvin-archastro
calvin-archastro deleted the fix/plan-collab-undo-status branch October 22, 2026 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant