Skip to content

Claude query parameters - #1410

Merged
eschultink merged 3 commits into
rc-v0.6.10from
claude_query_parameters
Sep 2, 2026
Merged

Claude query parameters#1410
eschultink merged 3 commits into
rc-v0.6.10from
claude_query_parameters

Conversation

@aperez-worklytics

@aperez-worklytics aperez-worklytics commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Rules updated after https://github.com/Worklytics/evalengin/pull/9028: including more query parameter in rules for Claude Compliance

Fixes

paste links to issues/tasks in project management

Features

Psoxy update

Logistics

paste links to issues/tasks in project management

Change implications

  • dependencies added/changed? no
  • something important to note in future release notes?
    • NOTE in CHANGELOG.md anything that will show up in terraform plan/apply that isn't
      obviously a no-op? no
    • breaking changes? if in module/example that is NOT marked alpha, requires major version
      change no

@aperez-worklytics aperez-worklytics self-assigned this Sep 2, 2026
Copilot AI lite review requested due to automatic review settings September 2, 2026 12:19

Copilot AI 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.

🟢 Approval recommended

The rule expansions are narrowly scoped and are validated with new tests covering both allowed and blocked query-parameter cases.

Pull request overview

This PR updates the Claude (Anthropic compliance API) sanitizer rules to allow additional query parameters needed for incremental/paginated discovery, and adds explicit tests to ensure the request allowlist behavior is enforced (allowed vs blocked query params).

Changes:

  • Allow order_by on /v1/compliance/apps/chats to support org-wide incremental polling by update time.
  • Allow updated_at.gte on /v1/compliance/apps/sessions/local to support “changed since last pass” polling.
  • Allow limit/page on /v1/compliance/organizations and add tests that verify unrecognized params are blocked.
File summaries
File Description
java/core/src/test/java/co/worklytics/psoxy/rules/anthropic/ClaudeTests.java Adds new invocation examples plus JUnit tests that assert query-param allowlisting blocks unknown params and permits newly required ones.
docs/sources/anthropic/claude/claude.yaml Extends allowedQueryParams for specific Claude compliance endpoints to include order_by, updated_at.gte, and limit/page.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Copilot AI review requested due to automatic review settings September 2, 2026 12:59
@aperez-worklytics
aperez-worklytics requested review from eschultink and removed request for Copilot September 2, 2026 13:00
@eschultink
eschultink merged commit 1e49d91 into rc-v0.6.10 Sep 2, 2026
120 checks passed
@eschultink
eschultink deleted the claude_query_parameters branch September 2, 2026 22:08
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.

3 participants