Skip to content

Reuse capability discovery metadata across fenced runs - #61

Merged
first-assist merged 3 commits into
mainfrom
fix/ai-capability-discovery-cache-20260915
Sep 15, 2026
Merged

first-assist merged 3 commits into
mainfrom
fix/ai-capability-discovery-cache-20260915

Conversation

@first-assist

@first-assist first-assist commented Sep 15, 2026

Copy link
Copy Markdown
Collaborator

A later fenced run previously lost its discovery candidates even when the capability catalogue was unchanged. An optional bounded process-local cache now lets the same principal and AI conversation describe a prior candidate without another model-driven discovery call.

Cached entries contain discovery metadata only. Describe rechecks current catalogue fingerprints, principal and policy before issuing a fresh run-bound handle; execution permissions and fences are unchanged. TTL is configurable from one second to seven days, with a bounded oldest-first search count. The default remains disabled. graphql-orm-ai is 0.98.1; no data or GraphQL migration is needed.

Validation: all 16 capability-delivery tests and 428 provider-enabled library tests passed (4 explicit live tests ignored). Added coverage for cross-run reuse, fresh execution fences, revocation, principal/conversation isolation, fingerprints, expiry, eviction and disabled defaults. Warnings-denied all-target Clippy and Rustdoc, PascalCase naming, compile-only PostgreSQL/MSSQL checks, documentation/release-policy checks and an explicit package SemVer comparison passed (196 compatibility checks).

This compatibility commit is based on 17d42f8 so consumers can adopt the cache without moving their existing ORM/auth dependency universe. Merge this PR with a merge commit to retain that tested compatibility revision. README, changelog, migration notes and generated workspace inventory are updated.

  • Documentation updated
  • No documentation impact

@first-assist
first-assist marked this pull request as ready for review September 15, 2026 02:39
@first-assist
first-assist merged commit ae95c78 into main Sep 15, 2026
10 checks passed
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