Skip to content

fix(ci): repair lanes broken by retrieval disclosure commit - #3090

Merged
ScriptedAlchemy merged 8 commits into
masterfrom
devin/1791183080-master-ci-fixes
Oct 5, 2026
Merged

ScriptedAlchemy merged 8 commits into
masterfrom
devin/1791183080-master-ci-fixes

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Fixes the CI lanes broken on master by 20175c2e6d feat(retrieval): disclose route, churn, misses, and co-change and f4b8918541 feat(index): report accurate generic ambiguity census: workspace Clippy (match_bool + items_after_test_module), Dashboard check:embedded (stale plugin/chatgpt-extension/embedded/server.mjs after the SDK regeneration), Linux core-contracts (CURSOR_PLUGIN_SKILLS missing the new routing-tracedecay bundled skill), and Linux core-storage (partitioned_codec_has_stable_bytes_and_round_trips — the ambiguity census changed the sealed manifest bytes without a SEALED_GENERATION_FORMAT_REVISION bump or a PARTITIONED_FORMAT_STATE_DIGEST update).

Motivation

Every open PR's merge-ref CI currently fails these lanes on master. The failures are unrelated to the PRs themselves and reproduce on origin/master a439cabecc.

Changes

  • crates/tracedecay-mcp/src/handlers/analysis/hotspots.rs: match churn_available { true => …, false => … } → if/else (clippy's suggested rewrite; behavior unchanged).
  • crates/tracedecay-agent-hosts/src/agents/cursor.rs: add "routing-tracedecay" to CURSOR_PLUGIN_SKILLS — plugin/skills/routing-tracedecay/SKILL.md is model-invocable (no disable-model-invocation), so the bundled-skill index test requires it.
  • plugin/chatgpt-extension/embedded/server.mjs: rebuilt via pnpm run build in plugin/chatgpt-extension so the embedded bundle matches the regenerated @tracedecay/sdk.
  • crates/tracedecay-code-index/src/production/generation_statistics.rs: ambiguous_name_drops gains skip_serializing_if = "Option::is_none" (None means "not measured" and stays absent on the wire, matching the decode-compat test); mod tests moved below production items for items_after_test_module.
  • crates/tracedecay-code-index/src/production/sealed_codec.rs: SEALED_GENERATION_FORMAT_REVISION_V1 20 → 21 — the census field changes the canonical manifest payload, and repo convention pairs a serialized-shape change with a revision bump (19 → 20 did the same).
  • crates/tracedecay-code-index/tests/code_index_suite/production_orchestration.rs: PARTITIONED_FORMAT_STATE_DIGEST pinned to the revision-21 canonical bytes.

Test plan

  • cargo clippy --workspace --all-targets -- -D warnings — clean
  • cargo test -p tracedecay-code-index --test code_index_suite — 205/205 (incl. partitioned_codec_has_stable_bytes_and_round_trips)
  • bash scripts/require-exact-test.sh cargo test -p tracedecay-agent-hosts --lib agents::cursor::tests::session_context_skill_index_matches_bundle_skills -- --exact — 1 passed
  • pnpm run check:embedded (in plugin/chatgpt-extension) — "embedded artifacts are current"
  • cargo fmt --all -- --check — clean

Checklist

  • No secrets, credentials, or .env files included
  • Breaking changes documented (sealed generation format revision bump forces a one-time rebuild, per the existing superseded-revision path)

Link to Devin session: https://app.devin.ai/sessions/1f0d1bf1427c4efcabbcdbd01e63cebc
Open in Devin Desktop: https://app.devin.ai/desktop/session/1f0d1bf1427c4efcabbcdbd01e63cebc?variant=devin
Requested by: @ScriptedAlchemy


Devin Review

ScriptedAlchemy and others added 3 commits October 5, 2026 00:34
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@changeset-bot

changeset-bot Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 0e26ada

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

ScriptedAlchemy and others added 3 commits October 5, 2026 00:48
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@ScriptedAlchemy
ScriptedAlchemy merged commit 48e105a into master Oct 5, 2026
13 of 15 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