Skip to content

Conversation

@monadoid
Copy link
Contributor

@monadoid monadoid commented Jan 16, 2026

fix: CI was pulling the entire stagehand repo and re-creating the node SEA binary. Now it just pulls the binary.

fix: No longer using a STAGEHAND_TAG var that we have to set in github. Instead, it just pulls the latest stagehand-server github release.

docs: Added docs


Summary by cubic

Speeds up and simplifies the PyPI publish workflow by downloading the prebuilt Stagehand server binary from the latest GitHub Release instead of cloning and rebuilding from source. Removes the stagehand_tag input and STAGEHAND_TAG repo var (auto-selects latest), drops Node/pnpm build steps, and adds RELEASE_WORKFLOWS.md documenting the release flow and required secrets.

Written for commit 0cd451c. Summary will update on new commits.

Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

2 issues found across 20 files (changes from recent commits).

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="examples/pydoll_tab_example.py">

<violation number="1">
P3: Fix the docstring typo (`ePage.getFrameTree`) so the CDP method name is correct.</violation>

<violation number="2">
P2: Ensure the session is ended before exiting early when `cdp_url` is missing to avoid leaking a remote session.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 2 files

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name=".github/workflows/publish-pypi.yml">

<violation number="1" location=".github/workflows/publish-pypi.yml:95">
P2: `RELEASE_TAG` includes `/` (stagehand-server/v…), but the GitHub API call uses it unencoded. This can turn the slash into a path separator and 404 the release lookup. URL-encode the tag before building the `/releases/tags/{tag}` URL.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

@monadoid monadoid merged commit 3bd7660 into main Jan 16, 2026
7 of 8 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.

2 participants