Add edb-proxy API endpoint with CORS support and request handling by Timidan · Pull Request #11 · Timidan/hexkit

Timidan · 2026-04-15T16:06:35Z

No description provided.

vercel · 2026-04-15T16:06:40Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
web3-toolkit	Ready	Preview, Comment	Apr 18, 2026 4:16am

chatgpt-codex-connector · 2026-04-15T16:06:40Z

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

Copilot

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds routing for an /api/edb proxy endpoint and updates request path extraction logic to rely on a rewrite-populated path query parameter.

Changes:

Added Vercel rewrites for /api/edb and /api/edb/* to forward to an edb-proxy handler.
Updated the EDB handler to compute subPath from req.query.path instead of parsing req.url.
Adjusted the relative import path for maybeInjectDefaultEtherscanKey.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
vercel.json	Adds rewrites intended to route `/api/edb` traffic through an `edb-proxy` endpoint.
api/edb/[...path].ts	Changes how the downstream sub-path is derived (now from `req.query.path`) and updates a relative import.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-04-15T16:11:37Z

+      "destination": "/api/edb-proxy"
+    },
+    {
+      "source": "/api/edb/:path*",
+      "destination": "/api/edb-proxy?path=:path*"


The rewrites send /api/edb and /api/edb/* requests to /api/edb-proxy, but the updated request handling logic is in api/edb/[...path].ts (which would normally be reached at /api/edb/*). With these rewrites in place, this handler won’t receive the traffic (and therefore won’t see req.query.path) unless there is a separate /api/edb-proxy serverless function that contains the same logic. Fix by either (a) changing the rewrite destination to /api/edb?path=:path* (or similar) so it hits this file, or (b) moving/duplicating this handler’s logic into the /api/edb-proxy function so the destination matches the implementation.

Suggested change

"destination": "/api/edb-proxy"

},

{

"source": "/api/edb/:path*",

"destination": "/api/edb-proxy?path=:path*"

"destination": "/api/edb?path="

},

{

"source": "/api/edb/:path*",

"destination": "/api/edb?path=:path*"

… key injection

Refactor and optimize code structure across multiple components

Copilot

Pull request overview

Copilot reviewed 65 out of 67 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-04-16T00:47:04Z

+export { ensureArray } from "./simulationArtifactTypes";
+import { ensureArray } from "./simulationArtifactTypes";


The module both re-exports ensureArray from ./simulationArtifactTypes and also imports the same symbol from the same module for local use. This is redundant and makes the export surface harder to follow. Prefer a single import and then re-export the imported binding (or import under a different name) so there’s only one linkage to ./simulationArtifactTypes.

@copilot apply changes based on this feedback

…ction

Copilot

Pull request overview

Copilot reviewed 69 out of 71 changed files in this pull request and generated 5 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-04-16T14:00:14Z

+        // Refine around the chosen Hook to find the best nearby one.
+        // Skip refinement when the eval-probe already confirmed Hook snapshots
+        // — they are verified to support eval and the refine batch (~51 RPCs)
+        // would just add latency without improving the result.
+        const evalProbeConfirmed = nonEntryHooks.length === 0 && hookIdsInRange.length > 0;
+        if (nearestFromTargeted !== null && step > 1 && !evalProbeConfirmed) {
+          const REFINE_WINDOW = 25;


evalProbeConfirmed is computed as nonEntryHooks.length === 0 && hookIdsInRange.length > 0, which is true even if the eval-probe did not run or found no better Hook snapshots. That can incorrectly skip the refinement batch and leave you with only entry-point Hooks (often empty locals). Consider tracking whether the eval-probe actually produced hookProbeIds.length > 0 and only skipping refinement in that case.

@copilot apply changes based on this feedback

Copilot · 2026-04-16T14:00:15Z

          console.error('[useDebugPrep] auto-connect failed:', err);
+          if (prepareIdRef.current === prepareId) {
+            setPrepState((prev) => ({
+              ...prev,
+              error: 'Live session was evicted before connection completed. Click Open Debugger to use trace-based debugging.',
+            }));
+          }


In the auto-connect failure path, prepState stays status: 'ready' while setting only error. This leaves a stale "ready" state even though no session is connected, and downstream logic (e.g., click handlers) may refuse to restart prep because it thinks prep is already ready. Consider setting status: 'failed' (or resetting to idle via cancel) and clearing sessionId/snapshotCount so users can retry preparation cleanly.

@copilot apply changes based on this feedback

Copilot · 2026-04-16T14:00:15Z

+    let preppedSessionDead = false;
+    if (
+      debugPrepForSimulation?.status === "ready" &&
+      debugPrepForSimulation.sessionId &&
+      debugSession?.sessionId !== debugPrepForSimulation.sessionId
+    ) {
+      try {
+        await connectToSession({
+          sessionId: debugPrepForSimulation.sessionId,
+          rpcPort: 0,
+          snapshotCount: debugPrepForSimulation.snapshotCount ?? 0,
+          chainId,
+          simulationId,
+        });
+        openDebugWindow();
+        return;
+      } catch (err) {
+        console.warn("[handleOpenDebug] Failed to connect to prepped session:", err);
+        preppedSessionDead = true;
+      }


If connecting to the "ready" prepped session fails, preppedSessionDead is set and later the code skips startReplayDebugPreparation entirely. This can strand users: the stale prepState remains ready, but no live session is usable and no new prep is started. A safer flow is to call cancelDebugPrep() on failure (or otherwise mark the prepState failed) and then allow startReplayDebugPreparation(simulationId) to run so a new live session can be prepared.

@copilot apply changes based on this feedback

Fixed in e70d6b6. When connecting to the ready prepped session fails, cancelDebugPrep() is now called to clear the stale ready state, and the !preppedSessionDead guard on startReplayDebugPreparation is removed so a fresh prep can be started automatically.

Copilot · 2026-04-16T14:00:16Z

+    if (debugRequested && !preppedSessionDead) {
      if (startReplayDebugPreparation(simulationId)) {
        showSuccess(


The debugRequested && !preppedSessionDead guard prevents retrying preparation when the previous ready session was evicted. Instead of skipping prep when preppedSessionDead is true, reset/cancel the stale prep state and proceed with preparation so "Open Debugger" can recover automatically.

@copilot apply changes based on this feedback

Already addressed in e70d6b6 — cancelDebugPrep() is called in the catch block to clear the stale prep state, and the !preppedSessionDead guard was removed from the if (debugRequested) check so startReplayDebugPreparation always runs when appropriate.

… in ExecutionStackTrace

…r predicate Agent-Logs-Url: https://github.com/Timidan/hexkit/sessions/770a26dc-858f-4df0-a6ba-71d03e39aa88 Co-authored-by: Timidan <25876969+Timidan@users.noreply.github.com>

…s dead Agent-Logs-Url: https://github.com/Timidan/hexkit/sessions/c9c0abc9-6d82-415a-90be-0fc87c8a4dc6 Co-authored-by: Timidan <25876969+Timidan@users.noreply.github.com>

feat: add edb-proxy API endpoint with CORS support and request handling

1d64986

Copilot AI review requested due to automatic review settings April 15, 2026 16:06

Copilot started reviewing on behalf of Timidan April 15, 2026 16:07 View session

vercel Bot deployed to Preview April 15, 2026 16:07 View deployment

Copilot AI reviewed Apr 15, 2026

View reviewed changes

Timidan added 2 commits April 15, 2026 19:02

Refactor and optimize code structure across multiple components

a1ddd99

feat: enhance debug session handling and proxy configuration with API…

5347d19

… key injection

vercel Bot deployed to Preview April 16, 2026 00:37 View deployment

Merge pull request #12 from Timidan/feat/cleanup

afc3b49

Refactor and optimize code structure across multiple components

Copilot AI review requested due to automatic review settings April 16, 2026 00:37

Copilot started reviewing on behalf of Timidan April 16, 2026 00:38 View session

vercel Bot deployed to Preview April 16, 2026 00:39 View deployment

Copilot AI reviewed Apr 16, 2026

View reviewed changes

feat: increase evaluation timeouts and enhance snapshot resolution

7b25870

vercel Bot had a problem deploying to Preview April 16, 2026 13:28 Failure

fix: update type definition for isOpcode in resolveEvalSnapshotId fun…

1fbab74

…ction

Copilot AI review requested due to automatic review settings April 16, 2026 13:51

Copilot started reviewing on behalf of Timidan April 16, 2026 13:52 View session

vercel Bot deployed to Preview April 16, 2026 13:52 View deployment

Copilot AI reviewed Apr 16, 2026

View reviewed changes

feat: enhance token movements handling and improve accordion behavior…

001ed3a

… in ExecutionStackTrace

vercel Bot deployed to Preview April 16, 2026 21:21 View deployment

Copilot started work on behalf of Timidan April 18, 2026 02:37 View session

fix: check r.value.value != null instead of r.value !== null in filte…

cdf9628

…r predicate Agent-Logs-Url: https://github.com/Timidan/hexkit/sessions/770a26dc-858f-4df0-a6ba-71d03e39aa88 Co-authored-by: Timidan <25876969+Timidan@users.noreply.github.com>

Copilot AI review requested due to automatic review settings April 18, 2026 02:39

Timidan review requested due to automatic review settings April 18, 2026 02:39

vercel Bot deployed to Preview April 18, 2026 02:40 View deployment

Copilot finished work on behalf of Timidan April 18, 2026 02:41

Copilot started work on behalf of Timidan April 18, 2026 04:13 View session

fix: cancel stale prep state and allow re-prep when prepped session i…

e70d6b6

…s dead Agent-Logs-Url: https://github.com/Timidan/hexkit/sessions/c9c0abc9-6d82-415a-90be-0fc87c8a4dc6 Co-authored-by: Timidan <25876969+Timidan@users.noreply.github.com>

Copilot AI review requested due to automatic review settings April 18, 2026 04:15

Timidan review requested due to automatic review settings April 18, 2026 04:15

vercel Bot deployed to Preview April 18, 2026 04:16 View deployment

Copilot finished work on behalf of Timidan April 18, 2026 04:17

Copilot started work on behalf of Timidan April 21, 2026 01:12 View session

Copilot finished work on behalf of Timidan April 21, 2026 01:13

Timidan merged commit 722cb4f into master Apr 22, 2026
3 checks passed

		export { ensureArray } from "./simulationArtifactTypes";
		import { ensureArray } from "./simulationArtifactTypes";

Conversation

Timidan commented Apr 15, 2026

Uh oh!

vercel Bot commented Apr 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

chatgpt-codex-connector Bot commented Apr 15, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Apr 15, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Apr 16, 2026

Choose a reason for hiding this comment

Uh oh!

Timidan Apr 21, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Apr 16, 2026

Choose a reason for hiding this comment

Uh oh!

Timidan Apr 21, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 16, 2026

Choose a reason for hiding this comment

Uh oh!

Timidan Apr 21, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 16, 2026

Choose a reason for hiding this comment

Uh oh!

Timidan Apr 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 16, 2026

Choose a reason for hiding this comment

Uh oh!

Timidan Apr 21, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 21, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

vercel Bot commented Apr 15, 2026 •

edited

Loading