Add runtime-extensible Cordis MCP server - #32
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7cfef07190
ℹ️ 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".
7cfef07 to
deaa2dc
Compare
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: deaa2dcd54
ℹ️ 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".
deaa2dc to
5081759
Compare
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9c1fabb520
ℹ️ 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".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 36ffe26f54
ℹ️ 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".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b41e5b472f
ℹ️ 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".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cfab0fb562
ℹ️ 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".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bc4e5ae97e
ℹ️ 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".
Replace mandatory alternatives with conditional, concrete improvements that identify a real issue and provide useful wording. Give flow and tone questions a direct, fidelity-preserving assessment and require behavioral coverage to include an actual revision. Add the decision-review skill from PR #32 with validation and discovery, and require delivery to wait for a started remote review of the exact final head before reporting completion. LLM-disclaimer: This commit was generated by an LLM.
Replace mandatory alternatives with conditional, concrete improvements that identify a real issue and provide useful wording. Give flow and tone questions a direct, fidelity-preserving assessment and require behavioral coverage to include an actual revision. Add the decision-review skill from PR #32 with validation and discovery, and require delivery to wait for a started remote review of the exact final head before reporting completion. LLM-disclaimer: This commit was generated by an LLM.
Replace mandatory alternatives with conditional, concrete improvements that identify a real issue and provide useful wording. Give flow and tone questions a direct, fidelity-preserving assessment and require behavioral coverage to include an actual revision. Add the decision-review skill from PR #32 with validation and discovery, and require delivery to wait for a started remote review of the exact final head before reporting completion. LLM-disclaimer: This commit was generated by an LLM.
Replace mandatory alternatives with conditional, concrete improvements that identify a real issue and provide useful wording. Give flow and tone questions a direct, fidelity-preserving assessment and require behavioral coverage to include an actual revision. Add the decision-review skill from PR #32 with validation and discovery, and require delivery to wait for a started remote review of the exact final head before reporting completion. LLM-disclaimer: This commit was generated by an LLM.
b9ead54 to
968be9e
Compare
|
@codex review Please review the exact current head LLM-disclaimer: This comment was generated by an LLM. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 968be9ec80
ℹ️ 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".
968be9e to
033470d
Compare
|
@codex review Please review the exact current head LLM-disclaimer: This comment was generated by an LLM. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 033470de15
ℹ️ 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".
033470d to
fb5c9ae
Compare
|
@codex review Please review the exact current head LLM-disclaimer: This comment was generated by an LLM. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: fb5c9ae179
ℹ️ 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".
fb5c9ae to
6187a54
Compare
|
@codex review Please review the exact current head LLM-disclaimer: This comment was generated by an LLM. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6187a5426d
ℹ️ 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".
6187a54 to
8cc124b
Compare
|
@codex review Please review the exact current head LLM-disclaimer: This comment was generated by an LLM. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8cc124b79e
ℹ️ 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".
Replace the custom worker/version-store prototype with a standalone stdio MCP that uses official Cordis Loader, Include, HMR, and Timer packages. Runtime entries are normal ESM modules backed by standard `cordis.yaml`: reusable modules live under `projects/mcp_cordis/plugins`, while disposable modules live under ignored `out/mcp_cordis` storage. The MCP exposes fixed discovery, lifecycle, invocation, promotion, and bounded execution gateways. It includes repository, Git worktree, and network starter modules, plus lifecycle, HMR rollback, restart, stdio isolation, timeout, and process-tree regression coverage. This branch also incorporates PR 24's scoped `projects/agents` skill changes. Final hosted-review corrections bind repository and Git operations to verified file or directory handles, bound permanent process-inspection failures, and make timeout cleanup coverage startup-safe. Regex search now fails closed when ripgrep is unavailable, avoiding unsafe backtracking and incompatible fallback semantics while retaining bounded fixed-string fallback search. Fixed-string fallback also fails closed for invalid UTF-8 so replacement decoding cannot corrupt raw byte offsets. Verification: - affected tests, skill links, and Buildifier: 12/12 passed - affected build: 29/29 targets passed - exact-current `git diff --check`: passed - exact-current Buildifier test: passed - exact-current `//projects/mcp_cordis:docs` build: passed - independent whole-diff review: accepted with no actionable findings LLM-disclaimer: This commit was generated by an LLM.
8cc124b to
e1ed4d4
Compare
|
@codex review Please review the exact current head LLM-disclaimer: This comment was generated by an LLM. |
|
Codex Review: Didn't find any major issues. 👍 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
2 participants
Replace the custom worker/version-store prototype with a standalone stdio MCP
that uses official Cordis Loader, Include, HMR, and Timer packages. Runtime
entries are normal ESM modules backed by standard
cordis.yaml: reusablemodules live under
projects/mcp_cordis/plugins, while disposable modules liveunder ignored
out/mcp_cordisstorage.The MCP exposes fixed discovery, lifecycle, invocation, promotion, and bounded
execution gateways. It includes repository, Git worktree, and network starter
modules, plus lifecycle, HMR rollback, restart, stdio isolation, timeout, and
process-tree regression coverage. This branch also incorporates PR 24's scoped
projects/agentsskill changes.Final hosted-review corrections bind repository and Git operations to verified
file or directory handles, bound permanent process-inspection failures, and
make timeout cleanup coverage startup-safe. Regex search now fails closed when
ripgrep is unavailable, avoiding unsafe backtracking and incompatible fallback
semantics while retaining bounded fixed-string fallback search.
Fixed-string fallback also fails closed for invalid UTF-8 so replacement
decoding cannot corrupt raw byte offsets.
Verification:
git diff --check: passed//projects/mcp_cordis:docsbuild: passedLLM-disclaimer: This pull request was generated by an LLM.