Skip to content

feat: add manifest process state grammar - #195

Merged
csfh merged 2 commits into
mainfrom
feature/manifest-process-state-grammar
Jun 9, 2026
Merged

feat: add manifest process state grammar#195
csfh merged 2 commits into
mainfrom
feature/manifest-process-state-grammar

Conversation

@csfh

@csfh csfh commented Jun 9, 2026

Copy link
Copy Markdown
Member

Closes #181
Closes #182
Closes #183
Closes #184

Summary

  • Adds canonical RuntimeStatus values to Direct Managed Process views and External Managed Process snapshots.
  • Updates Manifest rows and summaries to use compact process-state grammar.
  • Adds aggregate/scoped Process Output foundation and dynamic scope shortcut labels.
  • Initializes Badger Book workflow marker and Project delivery status docs.

Tests

  • bun run lint
  • bun run typecheck
  • bun run test
  • bun run format:check

Notes

  • Aggregate Process Output support is included as foundation for the later Process Output panel rework, but follow-up issues still cover ordering, interaction reset, and title polish.

Self-reviewed diff for scope, secrets, debug logs, and acceptance criteria.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 603253dc9a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/runtime-status.ts
Comment on lines +65 to +67
case "created":
case "removing":
return "starting";

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Treat created containers as off

When Docker Compose reports a service as created (and this adapter also assigns created to configured services with no ps entry in src/docker.ts), the container is not running rather than actively starting; Docker's ps docs list created separately from running, where running is the state after docker start/docker run (https://docs.docker.com/reference/cli/docker/container/ls/). Mapping it to starting makes never-started Compose services show Str indefinitely and skews panel summaries/colors until the user starts them.

Useful? React with 👍 / 👎.

@csfh

csfh commented Jun 9, 2026

Copy link
Copy Markdown
Member Author

Addressed Codex feedback: Docker/Compose now maps to , with regression coverage in . CI is green.

@csfh

csfh commented Jun 9, 2026

Copy link
Copy Markdown
Member Author

Addressed Codex feedback: Docker Compose created state now maps to off, with regression coverage in runtime-status.test.ts. CI is green.

@csfh
csfh merged commit 4e11434 into main Jun 9, 2026
3 checks passed
@csfh
csfh deleted the feature/manifest-process-state-grammar branch June 9, 2026 13:55
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.

Add Empty Manifest CTA Render Manifest State Summaries Render Manifest Rows With Compact State Grammar Land Canonical Runtime Process State

1 participant