Skip to content

fix: remove counters#243

Merged
idosal merged 7 commits into
mainfrom
remove-counter
May 8, 2026
Merged

fix: remove counters#243
idosal merged 7 commits into
mainfrom
remove-counter

Conversation

@idosal
Copy link
Copy Markdown
Owner

@idosal idosal commented May 8, 2026

No description provided.

Copilot AI review requested due to automatic review settings May 8, 2026 08:41
@idosal idosal requested review from Copilot and removed request for Copilot May 8, 2026 09:00
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 9 out of 10 changed files in this pull request and generated 2 comments.

Comment thread src/index.ts
Comment thread playwright.config.ts Outdated
idosal and others added 2 commits May 8, 2026 15:32
Pin @modelcontextprotocol/inspector@0.10.2 via the lockfile and invoke
it via pnpm exec in playwright.config.ts, replacing the per-run
npx fetch.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Wrap env.AI.autorag(...).search() in withAutoragCache(), keyed on a
SHA-256 of the pipeline + full search request. Cache API has no
per-op billing, so a hit skips the AutoRAG call entirely and a miss
costs the same as before. 12h TTL matches the existing KV cache TTL.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

The pinned inspector 0.10.2 didn't expose a "Streamable HTTP" transport
option, so every test waited 30s for a non-existent dropdown entry and
the job hit GitHub's 15-min timeout on retry. Bump to 0.21.2, pass
DANGEROUSLY_OMIT_AUTH=true so the test browser can reach the proxy
without a session token, and update the tool-input selector to match
the new "url*" required-field label. Also drop CI retries to 2 and add
the list reporter so CI logs stream per-test results.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@idosal idosal merged commit 7c96a4a into main May 8, 2026
5 checks passed
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