Skip to content

Add initial Markdown Output support - #2

Merged
jvmvik merged 8 commits into
devfrom
markdown
Oct 13, 2026
Merged

jvmvik merged 8 commits into
devfrom
markdown

Conversation

@strzibny

@strzibny strzibny commented Sep 10, 2026 •

Copy link
Copy Markdown
Contributor

Add support for searches with output=md

- Drop README.md.erb (removed on dev)
- Keep markdown() as an alias of md(); fix /markdown endpoint does not exist (404)
- Merge changelog entries
…1.2.1 notes

- raw() returns an error on non-2xx HTTP status instead of the error body
- search doctest reads $SERPAPI_KEY instead of a placeholder key
- Simplify search_archive test
- Ignore .env
@jvmvik jvmvik self-assigned this Oct 13, 2026
@jvmvik

jvmvik commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

Apologize, the version 1.2.0 was the first fully Opus 5.5 driven release, and it went bad.
The instruction was to replicate the changes from serpapi-ruby, but it left out all the PR aside, and release promptly
A markdown was mapped /markdown, which is totally wrong, and the test had 2 layers of mistakes: http code unchecked, output string unverified
@strzibny All your changes have been integrated now. can you cross review ?

@strzibny

Copy link
Copy Markdown
Contributor Author

@jvmvik Since markdown() as a method was not working before, I think we can remove it instead of keeping it an alias (less surface, less confusion, less future bugs). I would just keep md, but not a blocker.

@jvmvik

jvmvik commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

Agree. keep it sync with the reference serpapi-rub, and documentation.

@jvmvik
jvmvik merged commit 80a625c into dev Oct 13, 2026
1 check 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