Skip to content

validate: skip the vendored SDK - #54

Merged
manish-wekan merged 1 commit into
mainfrom
fix/validate-skip-vendored-sdk
Oct 16, 2026
Merged

manish-wekan merged 1 commit into
mainfrom
fix/validate-skip-vendored-sdk

Conversation

@manish-wekan

Copy link
Copy Markdown
Collaborator

nitrostack-py validate walked vendor/nitrostack (the SDK copy init vendors) as project code. nitrostack/testing applies @mcp_app to a class defined inside a function, so validate_mcp_app_imports reported:

[ERROR] vendor/nitrostack/nitrostack/testing/__init__.py: '...' uses @mcp_app but no decorated application class was found after import.

NitroStudio Compose runs validate as its Python typecheck, so a build could not get past it: the agent cannot fix SDK code and went idle (found by the Studio edge-case bench, Python arm).

The validator now skips the vendored SDK directory (VENDOR_REL) only; other vendor/ content is still checked.

Tests

  • New test_validate_skips_the_vendored_sdk fails on main with the exact error above and passes with the fix.
  • Full suite passes.
  • On the failed bench project, validate_project now returns no issues.

🤖 Generated with Claude Code

`init` vendors the SDK into vendor/nitrostack. `validate` walked it as
project code, and nitrostack/testing applies @mcp_app to a class defined
inside a function, so every vendored project that imported
nitrostack.testing failed with "uses @mcp_app but no decorated
application class was found after import". Compose's typecheck runs
`validate`, so a NitroStudio build could not get past it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@manish-wekan
manish-wekan merged commit 7dafac9 into main Oct 16, 2026
3 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