Skip to content

fix(deploy): restore shared-deployment namespace merge (src/__init__.py collision) - #825

Merged
blove merged 1 commit into
mainfrom
blove/shared-deploy-src-collision
Aug 18, 2026
Merged

fix(deploy): restore shared-deployment namespace merge (src/__init__.py collision)#825
blove merged 1 commit into
mainfrom
blove/shared-deploy-src-collision

Conversation

@blove

@blove blove commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Production smoke has been failing with all-graph run timeouts. Root cause (via LangGraph Cloud revision logs): every revision since Aug 7 is DEPLOY_FAILED with Graph 'c-interrupts' failed to load: ModuleNotFoundError: No module named 'src.aviation_tools' — repeated failed rollouts today then wedged the run queue on the old revision.

cockpit/langgraph/client-tools/python/src/__init__.py (an empty license-header file shipped in #642) turned that dep's src into a regular package. Per PEP 420 a regular package wins exclusively over all namespace portions, so on the shared deployment's aggregated sys.path every other dep's from src.x import ... stopped resolving. Reproduced locally against the staged deps/ tree; after deleting the file, src.aviation_tools + src.graph import cleanly across the merged namespace.

  • delete the __init__.py (nothing imports src as a regular package; graph entrypoints load by file path)
  • generate-shared-deployment-config.ts now throws at manifest-generation time if any staged dep reintroduces a src/__init__.py, so this class of breakage fails in CI instead of at LangGraph Cloud startup

Merging this touches the deploy workflow's trigger paths, so Deploy LangGraph will roll a fresh revision; production smoke should recover once it's live.

🤖 Generated with Claude Code

…__init__.py

cockpit/langgraph/client-tools shipped an empty src/__init__.py (#642) that
turned its src into a regular package; per PEP 420 that wins over every other
dep's namespace portion, so c-interrupts' 'from src.aviation_tools import'
raised ModuleNotFoundError at startup and every revision since Aug 7 hit
DEPLOY_FAILED (production smoke timeouts were the wedged run queue). Delete
the file and make the manifest generator throw if any staged dep reintroduces
one.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@vercel

vercel Bot commented Aug 17, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
threadplane Ready Ready Preview Aug 17, 2026 9:02pm

Request Review

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Automated approval: this PR received an intelligent (AI) code review. See the review comments on this PR.

@github-actions

github-actions Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

@blove
blove merged commit 7038b05 into main Aug 18, 2026
116 of 123 checks passed
@blove
blove deleted the blove/shared-deploy-src-collision branch August 18, 2026 03:42
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