Skip to content

Route memories to saved project workspaces and simplify agent setup - #239

Closed
Coding-Dev-Tools wants to merge 35 commits into
mainfrom
codex/workspace-organization-pr
Closed

Coding-Dev-Tools wants to merge 35 commits into
mainfrom
codex/workspace-organization-pr

Conversation

@Coding-Dev-Tools

@Coding-Dev-Tools Coding-Dev-Tools commented Sep 28, 2026 •

Copy link
Copy Markdown
Owner

Agents now follow saved project-to-workspace mappings and inherit an authorized session destination when no workspace is supplied. Responses identify the selected destination. Command Code's SessionStart hook follows the nearest Git repository, and a previewed selective-move workflow organizes mixed workspaces while retaining history and workspace permissions.

Relocation policy uses domain operations declared in the core storage protocol. The SQLite adapter owns traversal and writes, bounds materialization with SQL limits, and preserves caller-owned transaction rollback. A 500-memory move uses 501 parameters, remaining compatible with SQLite's older 999-variable limit. An independent in-memory adapter exercises the same policy without a database connection.

The updater regression proves descendant cleanup with a real post-return probe. CodeQL runs for stacked codex/** PR bases, and the Pi development-host dependency audit includes the patched WebSocket client.

Stack: #238 → this PR → #240. All 29 required/current GitHub checks passed at e00581e517061504f26f850ff9acc7cf2db82105; no unresolved review conversations remain.

Validation for the current source:

  • Relocation/routing review: 300 focused cases passed. The SQLite follow-up passed 116 adjacent cases, including four regressions that failed before the repair. Final protocol module: 15 passed, with Python 3.9-compatible annotation inspection.
  • 100 evidence, documentation and renderer cases passed. Immutable v103 reruns the fixtures with unchanged aggregates; all 581 source hashes match staged Git blobs and the artifact checksum is verified.
  • Clarified explicit-workspace unknown-session recall and updated its shipped skill checksum. All 28 package/skill-package tests passed.
  • Pinned Pyright 1.1.414, Ruff, commercial manifest, dashboard asset and whitespace checks passed. Independent review found no remaining blocker in the protocol or SQLite repair.

These selections overlap. Earlier updater, browser and installed-client checks are retained in the PR history; they are not substituted for current-head CI. No release, deployment or provider call is included.

@Coding-Dev-Tools
Coding-Dev-Tools marked this pull request as ready for review September 28, 2026 08:07
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-29T00:49:08.472650Z 4fd41bc New commits
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@Coding-Dev-Tools
Coding-Dev-Tools changed the base branch from main to codex/review-48h-20260928 September 28, 2026 09:23

@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: 6b0537cea4

ℹ️ 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 engraphis/core/relocation.py Outdated

@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: 3af569ca27

ℹ️ 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 engraphis/core/store.py Outdated

@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: 4fd41bc43e

ℹ️ 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 skills/engraphis-memory/references/SCOPING.md Outdated
claude and others added 4 commits September 30, 2026 02:24
…ures

npm published three advisories after this stack last passed CI, failing both
Pi extension jobs introduced by the full development-tree audit:

- fast-uri <=3.1.7 and ip-address <=10.7.0 are production dependencies of
  the MCP SDK. Update the Pi lockfile to fast-uri 3.1.8 and ip-address 10.7.2.
- brace-expansion <=5.0.11 exists only inside the Pi coding agent used as the
  CI test host. That package ships its own npm-shrinkwrap.json, so npm
  overrides cannot replace it, and no Pi release (through 0.99.1) has a fixed
  lockfile yet.

Production dependencies stay strictly audited. Development advisories still
fail, except those confined to the test host's own lockfile, which are
reported as warnings until a fixed host can be adopted.

The step now runs under bash on both runners. On Windows the default pwsh
wrapper reported only the final command's exit code, so a failing npm ci,
verify or integration run was hidden whenever the audit passed.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014ksPGXPZQTikJYpaGHa6R6
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014ksPGXPZQTikJYpaGHa6R6
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

1 similar comment
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

1 similar comment
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

1 similar comment
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@Coding-Dev-Tools
Coding-Dev-Tools changed the base branch from codex/review-48h-20260928 to main October 1, 2026 09:33
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.

2 participants