Skip to content

docs: remove oauth_pkce_example.py, which imports helpers that no longer exist - #1015

Open
LukasParke wants to merge 1 commit into
mainfrom
devin/1791237874-readme-examples
Open

LukasParke wants to merge 1 commit into
mainfrom
devin/1791237874-readme-examples

Conversation

@LukasParke

@LukasParke LukasParke commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Summary

examples/oauth_pkce_example.py imports oauth_create_sha256_code_challenge, oauth_create_authorization_url, and the matching request types from openrouter.utils. Those helpers were removed in e6b0242 by a Speakeasy regeneration, so the example fails with ImportError (the old helper also built the wrong authorize URL). It was the only file in examples/ and nothing else in the repo (README, docs, pyproject, workflows) references it, so this PR deletes it instead of rewriting it against client.o_auth.

The README snippet issues (#589, #591) are out of scope here and are being handled in separate PRs.

Rebased onto current main, so CI includes the pytest step added in #1014.

Fixes #583

Link to Devin session: https://openrouter.devinenterprise.com/sessions/78072f08dc274fda9d8309045b1ab806
Requested by: @LukasParke

🤖 Generated with Claude Code

@devin-ai-integration

Copy link
Copy Markdown
Contributor

I'll fix CI failures and address comments from users with write access that start with 'Devin'.

  • Disable automatic comment, CI, and merge conflict monitoring

Original prompt from Luke

SYSTEM:
<latest_message>
Luke Parke (U0BF2BKT52N) [ts=1791235775.813579]: @Devin audit all open issues on our public Typescript, Python, and Go SDKs, and the terraform Provider repo, and list out the unique problems, and vet them to determine validity and severity
</latest_message>

=== BEGIN THREAD HISTORY (in #agents) ===
Luke Parke (U0BF2BKT52N) [ts=1791235775.813579]: @Devin audit all open issues on our public Typescript, Python, and Go SDKs, and the terraform Provider repo, and list out the unique problems, and vet them to determine validity and severity
=== END THREAD HISTORY ===
Channel ID: C07UF9XLTFF
Thread URL: https://openrouter.slack.com/archives/C07UF9XLTFF/p1791235775813579?thread_ts=1791235775.813579&amp;cid=C07UF9XLTFF

The <latest_message> is the message that you should use to guide your goals + task for this session, and you should use the rest of the slack thread as context.
A [ts=...] marker on a Slack message is that message's timestamp. To act on a specific message with the slack tool (e.g. adding an emoji reaction via the reaction command), pass that value as timestamp along with the Channel ID — no extra lookup call is needed.

@LukasParke
LukasParke force-pushed the devin/1791237874-readme-examples branch from c96ec6d to d9d3ed7 Compare October 21, 2026 17:46
@LukasParke
LukasParke marked this pull request as ready for review October 21, 2026 17:46

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

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.

examples/oauth_pkce_example.py is broken: it imports OAuth PKCE helpers that were deleted in e6b0242

1 participant