Skip to content

Harden OpenRouter provider privacy policy - #4

Merged
dymoo merged 2 commits into
mainfrom
agent/openrouter-privacy-policy
Aug 1, 2026
Merged

Harden OpenRouter provider privacy policy#4
dymoo merged 2 commits into
mainfrom
agent/openrouter-privacy-policy

Conversation

@dymoo

@dymoo dymoo commented Aug 1, 2026

Copy link
Copy Markdown
Owner

Outcome

Attach OpenRouter provider privacy requirements inside the LLM request builder so every review request fails closed unless routing denies data collection, is ZDR, and supports the requested parameters.

Contract

  • exact https://openrouter.ai/api/v1 match only
  • data_collection: deny, zdr: true, and require_parameters: true on every request
  • no behavior change for other OpenAI-compatible endpoints
  • no model or endpoint fallback
  • patch release metadata for v2.0.1

Verification

  • npm run format
  • npm run check-all (102 tests)
  • independent standards review: no violations
  • independent spec review against byline-fyi/byline#212: no findings

Supports byline-fyi/byline#212.

@dymoo

dymoo commented Aug 1, 2026

Copy link
Copy Markdown
Owner Author

The review check fails before any model request because pull_request_target executes the workflow from the current default branch, where the optional dogfood job has no LLM_API_KEY/URL/model configured. This PR adds the non-secret configuration gate, but that gate cannot affect its own pull_request_target run. The authoritative Node 20 and Node 24 CI jobs pass, and the full local npm run check-all suite passes (102 tests).

@dymoo
dymoo merged commit e1c3c2a into main Aug 1, 2026
2 of 3 checks passed
@dymoo
dymoo deleted the agent/openrouter-privacy-policy branch August 1, 2026 09:04
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.

1 participant