Repository navigation
test: validate an isolated native Codex MemoryD instance - #266
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. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4e5bbfcbc3
ℹ️ 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".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 60e96e66bf
ℹ️ 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".
Summary
--host/--portoverrides preserve the unspecified component of an inherited bind while replacing stale persisted endpoints.MEMORYD_CODEX_PORT, add matching--bindto runbook initialization, and fix the recall jq filter to read.data // ..waitpid(WNOHANG); the timeout starts when shutdown is requested.PRs #261–#265 were checked for overlap. No directly matching issue was found, so no issue was created or linked.
Validation
cargo test --test native_instance_isolation -- --nocapture— 4 passed.scripts/v0.1-release-gate.sh --artifact-dir /tmp/codex-memoryd-dakota-review-two-findings— passed: 572 tests across 48 result groups, zero failures; formatting, whitespace, release build, doctor, eval, performance, and dry-run checks passed.Safety boundary
All test records and native memory files are synthetic and temporary. No Dakota access or deployment, Friday service/store/config/port/systemd change, private-data transfer, cloud migration, Devsy change, database reset, provider writeback, or Codex config mutation was performed. Native Codex memory remains available as fallback.