Skip to content

Conversation

miguelg719
Copy link
Collaborator

why

Used to be possible, now it's breaking

what changed

Set default schema whenever extract options doesn't provide one

test plan

@miguelg719 miguelg719 marked this pull request as ready for review July 3, 2025 20:31
Copy link

changeset-bot bot commented Jul 3, 2025

🦋 Changeset detected

Latest commit: bfc4b84

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

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

@miguelg719 miguelg719 added the extract These changes pertain to the extract function label Jul 3, 2025
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

Fixes backward compatibility by restoring default schema behavior in extract method when no schema is provided in options.

  • Modified lib/StagehandPage.ts to set default extract schema when options are provided without one, preventing breaking changes
  • Maintains backward compatibility with existing code that relied on implicit default schema behavior

1 file reviewed, no comments
Edit PR Review Bot Settings | Greptile

@miguelg719 miguelg719 merged commit b077d3f into main Jul 3, 2025
16 of 27 checks passed
seanmcguire12 pushed a commit that referenced this pull request Jul 4, 2025
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @browserbasehq/[email protected]

### Patch Changes

- [#856](#856)
[`8a43c5a`](8a43c5a)
Thanks [@seanmcguire12](https://github.com/seanmcguire12)! - set
download behaviour by default

- [#857](#857)
[`890ffcc`](890ffcc)
Thanks [@miguelg719](https://github.com/miguelg719)! - return
"not-supported" for elements inside the shadow-dom

- [#844](#844)
[`64c1072`](64c1072)
Thanks [@seanmcguire12](https://github.com/seanmcguire12)! - don't
automatically close tabs

- [#860](#860)
[`b077d3f`](b077d3f)
Thanks [@miguelg719](https://github.com/miguelg719)! - Set default
schema on extract options with no schema

- [#842](#842)
[`8bcb5d7`](8bcb5d7)
Thanks [@seanmcguire12](https://github.com/seanmcguire12)! - improved
handling for OS level dropdowns

- [#846](#846)
[`7bf10c5`](7bf10c5)
Thanks [@miguelg719](https://github.com/miguelg719)! - Filter attaching
to target worker / shared_worker

## @browserbasehq/[email protected]

### Patch Changes

- Updated dependencies
\[[`8a43c5a`](8a43c5a),
[`890ffcc`](890ffcc),
[`64c1072`](64c1072),
[`b077d3f`](b077d3f),
[`8bcb5d7`](8bcb5d7),
[`7bf10c5`](7bf10c5)]:
    -   @browserbasehq/[email protected]

## @browserbasehq/[email protected]

### Patch Changes

- Updated dependencies
\[[`8a43c5a`](8a43c5a),
[`890ffcc`](890ffcc),
[`64c1072`](64c1072),
[`b077d3f`](b077d3f),
[`8bcb5d7`](8bcb5d7),
[`7bf10c5`](7bf10c5)]:
    -   @browserbasehq/[email protected]

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
extract These changes pertain to the extract function
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants