Skip to content

feat: add pane command registry core#186

Open
parsakhaz wants to merge 1 commit into
phase2-local-daemon-protocolfrom
phase2-command-registry-core
Open

feat: add pane command registry core#186
parsakhaz wants to merge 1 commit into
phase2-local-daemon-protocolfrom
phase2-command-registry-core

Conversation

@parsakhaz
Copy link
Copy Markdown
Member

Summary

  • add a daemon command registry with IPC binding tests for daemon-owned channels
  • route folder, session-log, and resource monitor IPC handlers through the shared registry
  • replace remaining folder and panel event direct window sends in this slice with the runtime event sink

Testing

  • pnpm typecheck
  • pnpm lint
  • CI=1 pnpm --filter main test
  • PANE_DIR=/tmp/pane-phase2-command-registry-core xvfb-run -a pnpm test:ci

@parsakhaz
Copy link
Copy Markdown
Member Author

@codex review

@parsakhaz
Copy link
Copy Markdown
Member Author

@claude review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Something went wrong. Try again later by commenting “@codex review”.

We were unable to download your code in a timely manner.
ℹ️ 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".

@parsakhaz
Copy link
Copy Markdown
Member Author

@codex review

@parsakhaz
Copy link
Copy Markdown
Member Author

@claude review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Already looking forward to the next diff.

ℹ️ 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".

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