Skip to content

Forward-port SDK 2.1.0 changes to master - #56

Merged
TheRealAgentK merged 4 commits into
masterfrom
forward-port/2.1.0
Nov 1, 2026
Merged

TheRealAgentK merged 4 commits into
masterfrom
forward-port/2.1.0

Conversation

@TheRealAgentK

Copy link
Copy Markdown
Collaborator

Summary

Forward-port the applicable changes from the released SDK 2.1.0 (#55) without merging the release branch into master.

  • Preserve response headers on HTTPError and RateLimitError, with regression tests.
  • Enable CI for release/** and support/** branches alongside master.
  • Document supported-version releases and selective forward-porting.
  • Record the 2.1.0 release notes and regenerate API docs from the combined master source.

Preserved development behavior

  • Both version declarations remain 3.0.0.dev0; the 2.1.0 version bump is excluded.
  • Master retains its default Autohive User-Agent feature and its existing tests. The 2.x User-Agent revert is not imported.
  • Includes the earlier ci/support-branch-tests changes; no separate CI PR is needed.

Verification

  • Full SDK suite on Python 3.14.7: 104 passed, 100% coverage, including the existing User-Agent tests.
  • Verified development version declarations and regenerated API documentation contain the error headers API and User-Agent functionality.
  • Source, test, workflow, and release-note whitespace checks passed. Raw pdoc output retains generator-emitted trailing whitespace.

Refs #54

TheRealAgentK and others added 4 commits November 2, 2026 10:22
ExecutionContext.fetch() exposed headers on successful responses but
discarded them when raising. Integrations such as JobAdder need provider
headers (for example X-Allow-Duplicates on HTTP 409) to handle failures.

- Add an optional keyword-only `headers` argument and attribute to
  HTTPError, defaulting to an empty dict.
- Pass response headers from fetch() for non-2xx responses and for 429.
- Keep the exception message, positional arguments, retry_after and
  retry behaviour unchanged, so headers never reach str(exc).
- Add release notes and regenerate API docs.

Refs #54
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 1, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-01T22:10:17.445732Z 2c00aa3 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@github-actions

github-actions Bot commented Nov 1, 2026

Copy link
Copy Markdown

Coverage — 2c00aa3 (feat: preserve response headers on HTTPError and RateLimitError) by @TheRealAgentK

Total coverage: 100%

File Stmts Miss Cover Missing
src/autohive_integrations_sdk/__init__.py 2 0 100%
src/autohive_integrations_sdk/integration.py 390 0 100%

@TheRealAgentK
TheRealAgentK merged commit 5969e1d into master Nov 1, 2026
1 check passed
@TheRealAgentK
TheRealAgentK deleted the forward-port/2.1.0 branch November 1, 2026 22:14
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