Skip to content

feat: add Go bindings - #30

Open
RemcoE33 wants to merge 3 commits into
firecrawl:mainfrom
RemcoE33:feat/go-bindings
Open

feat: add Go bindings#30
RemcoE33 wants to merge 3 commits into
firecrawl:mainfrom
RemcoE33:feat/go-bindings

Conversation

@RemcoE33

@RemcoE33 RemcoE33 commented Aug 6, 2026

Copy link
Copy Markdown

Summary

  • add cgo-backed Go document conversion library and installable anydoc CLI
  • add Go CI plus maintainer-only archive preparation and go/v* verification workflows
  • document the subdirectory Go module and release-preparation flow

Verification

  • go test -count=1 ./...
  • go vet ./...
  • cargo fmt --all --check
  • cargo build -p anydoc-go --release
  • sh scripts/check-versions.sh

Follow-up

A maintainer should run Prepare Go Archives after merge, commit its go-libraries artifact in a release-prep PR, then tag go/vX.Y.Z.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

# Conflicts:
#	.github/workflows/release.yml
#	.gitignore
#	Cargo.lock
#	Cargo.toml
#	README.md
#	scripts/check-versions.sh
@xusenlin

xusenlin commented Aug 8, 2026

Copy link
Copy Markdown

Worth flagging for this PR given #67: cgo puts lopdf's uncatchable SIGABRT
(RUSTSEC-2026-0187) in the same process as the caller's Go runtime, so a
~21 KB hostile PDF takes the whole service down rather than failing one
conversion. Might be worth a note in the README until pdf-inspector ships
the lopdf 0.42 bump.

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