Skip to content

Reuse path resolution within each registry inspection - #8416

Open
loopx-agent wants to merge 1 commit into
mainfrom
codex/registry-inspection-path-reuse-20261109
Open

loopx-agent wants to merge 1 commit into
mainfrom
codex/registry-inspection-path-reuse-20261109

Conversation

@loopx-agent

Copy link
Copy Markdown
Collaborator

Problem

A scoped native status read validates its registry even when the caller only needs to confirm the source of a saved citation. The inspector resolves the same repository and state-file paths once to check which Goals share them and again when it builds the response.

Change

Reuse each exact path spelling within one inspection. The shared-path checks and the reported absolute paths use the same observation; the next inspection resolves paths again. The existing filesystem resolver and native source validation remain in place.

Validation

  • On 15 paired cold citation reads over disposable native SQLite snapshots, median response time fell from 602 ms to 515 ms. All excerpt bytes and source status facts matched. Malformed source registration and source registry data still returned 503, and restoring them returned the original excerpt.
  • The scoped status path resolved four paths before and two after, on both File and SQLite. The alias regression confirms that a shared-state conflict still reports the physical path, and retargeting the alias changes the result on the next inspection.
  • 111 focused tests, Ruff, the configured mypy scope, exact-diff change quality and standard premerge passed. An isolated wheel served the workspace and read the same original citation, including refusal and recovery through the packaged source CLI.

The performance observation applies when the native source CLI includes this change. Existing sources pinned to an older CLI retain that version.

Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>

This branch has not been deployed

No deployments
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