Skip to content

Fix profile resolution inspection verdict#111

Merged
cbusillo merged 1 commit into
mainfrom
fix/profile-resolution-reason-head
Jun 19, 2026
Merged

Fix profile resolution inspection verdict#111
cbusillo merged 1 commit into
mainfrom
fix/profile-resolution-reason-head

Conversation

@cbusillo

Copy link
Copy Markdown
Owner

Summary

  • Preserve profile-resolution failures as profile_resolution_error instead of helper_plugin_error
  • Add regression assertions for missing and unreadable explicit inspection profiles
  • Update helper-facing docs and dogfood scripts to use inspect/inspect-closeout command names

Validation

  • JAVA_HOME=$(/usr/libexec/java_home -v 21) ./gradlew :test --tests com.shiny.inspectionmcp.InspectionHandlerTest
  • ./scripts/test-red-lane-smoke-script.sh
  • commit hook: ./gradlew :test, :inspection-core:test, :mcp-server-jvm:test, buildPlugin

Fixes the active auto-review P1 for explicit profile lookup failures.

@cbusillo cbusillo merged commit eec070e into main Jun 19, 2026
4 checks passed
@cbusillo cbusillo deleted the fix/profile-resolution-reason-head branch June 19, 2026 01:16
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