Repository navigation
fix: integrate project intelligence across agents and environments - #19
Open
ScriptedAlchemy wants to merge 6 commits into
Open
ScriptedAlchemy wants to merge 6 commits into
ScriptedAlchemy wants to merge 6 commits into
Conversation
|
Keep Loom available through the app while removing its MCP registration, provider allowlist entry, shared agent guidance, and replay scenario. Record the smaller measured agent context.
Use the active native conversation when budgeting knowledge steering after provider-session recovery. Preserve handoff membership and exclude the pending steer message from historical occupancy. Validate dedicated review-guide models for generateReviewGuide across every selected environment, keeping valid preferences or selecting a shared capable fallback. Integrate current main, resolve the context-budget baseline from actual measurements, and align Loom and settings fixtures with current client APIs. Preserve the budget assertion and approval workflow. Validated with the required changed-file gate: 1,313 tests across 43 files and typechecks for 32 packages. Regression cases failed before the corresponding fixes.
…e registry The registry replaced each snapshot's supportsTextGeneration with the static operation list, so an Antigravity profile with global hooks, or an uninstalled or signed-out instance, still advertised text generation and writers stopped falling back to the default model. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
Project intelligence existed in separate UI, MCP, provider, and delivery paths. That let active steering bypass knowledge resolution, retries depend on a paginated inspector, remote identifiers lose their environment, and provider settings offer operations a selected provider could not execute.
Changes
(attempt_id, message_id), preserve existing data, and keep forgetting and maintenance cursors correct for multiple messages in an attempt.Includes current main (
71206f5f4e34), preserving PR #18's linked-environment gates, remote approvals, and context-exhaustion recovery.Review follow-up fixes budget knowledge steering against the active native conversation after resume recovery, without counting the pending message twice. Dedicated review-guide models now require
generateReviewGuidesupport in every targeted environment and select a shared capable fallback when necessary. Loom and settings fixtures also follow the current client APIs.Validation
On Node 26.12.1:
Final review and current-main integration:
vp run check:changed --serialpassed the native project-memory addon build, format, lint, both knip passes, GraphQL consistency, typechecking for 32 affected packages, and 1,313 tests across 43 files, including Chromium settings regressions. The native-history and model-selection regressions failed before their fixes. The final working tree andgit diff --checkwere clean.Initial integration:
vp run check:changed --serialpassed format, lint, both knip passes, generated GraphQL, typechecking for 32 affected packages, and 632 tests across 38 files (including automated Chromium tests).Loom-tool removal:
vp run check:changed HEAD --serialpassed format, lint, both knip passes, generated GraphQL, typechecking for 23 affected packages, the native project-memory addon build, and 299 tests across 5 files, including the exact context measurement.Start/control/provider-switch integration: 123/123 passed after rebasing onto PR fix: merge upstream agent links and open remote approvals #18.
Agent replay graders: 13/13 passed; all 6 remaining intelligence cases construct valid Claude and Codex dry-run requests, with no Loom tool in either payload.
Exact agent-context measurement regression: 1/1 passed after recording measured sizes; the measurement assertion and workflow remain unchanged.
CI follow-up: updated both preview-upgrade migration expectations to include migration 81. All 5 preview-upgrade tests pass, including rollback/retry and import preservation.
vp run check:changed HEAD --serialalso passed format, lint, both knip passes, generated GraphQL, typechecking for 3 affected packages, and the native addon build.Additional verification includes migration rollback/data preservation, real SQLite delivery and restart cases, multi-environment authorization and navigation, provider capability enforcement, summary coverage beyond 500 records, and React-compiler-compatible mobile subscriptions.
Agent context measurements
Removing the Loom tool saves 1,650 bytes of MCP tool schemas and 105 bytes from each shared instruction payload compared with the first PR revision. The exact measurement regression records these UTF-8 byte counts:
The maintainer-approved
agent-context-increaselabel is applied for the remaining instruction growth; the measurement assertion and approval workflow are unchanged. Registered tool-schema context is smaller than main. These are byte counts, not token estimates.Limits
Live model evals were not run because provider API keys are unavailable; replay request construction, schemas, and graders were checked. The complete Windows installer has not been built on this Linux host; its target-selection regression is tested and Windows CI remains the native validation.
Implemented and verified with GPT-6 Astra Pro in the Codex harness.