Skip to content

Feat/extract api - #45

Open
adarshdigievo wants to merge 2 commits into
masterfrom
feat/extract-api
Open

adarshdigievo wants to merge 2 commits into
masterfrom
feat/extract-api

Conversation

@adarshdigievo

Copy link
Copy Markdown
Member
  • Add a dedicated client.extract method for the new Extract API
  • Update CI to test for Python 3.15

Copilot AI balanced review requested due to automatic review settings October 15, 2026 10:05

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

Static review found no blocking issues in the focused client addition and supporting changes; tests were not executed.

0 open findings

What changed in this PR

Adds Extract API support through the existing search client and extends CI coverage to Python 3.15.

Changes:

  • Adds client-level and module-level extract() methods.
  • Adds mocked and live tests, usage examples, and documentation.
  • Updates Python compatibility metadata and CI matrices.
File Description
tests/​test_extract.py Tests extraction parameters, outputs, and errors.
tests/​test_extract_live.py Adds live Extract API checks.
serpapi/​core.py Adds and exports extract().
README.md Documents extraction and refreshes search examples.
pyproject.toml Adds the Python 3.15 classifier.
docs/​user_guide/​index.md Links the Extract API guide.
docs/​user_guide/​extract.md Documents extraction modes, outputs, and options.
docs/​index.md Adds extraction to documentation navigation.
CONTRIBUTING.md Updates testing instructions and Python coverage.
.github/​workflows/​release.yml Adds Python 3.15 and moves example execution to it.
.github/​workflows/​ci.yml Adds Python 3.15 and moves example execution to it.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

This branch has not been deployed

No deployments
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