Skip to content

perf(server): index projected thread lookups - #199

Merged
leoisadev1 merged 4 commits into
mainfrom
perf/server-projector-index
Sep 8, 2026
Merged

perf(server): index projected thread lookups#199
leoisadev1 merged 4 commits into
mainfrom
perf/server-projector-index

Conversation

@leoisadev1

Copy link
Copy Markdown
Member

Problem

Projector updates and command invariants repeatedly scan the projected thread array to find the same record. Large environments pay that lookup cost for each update.

Changes

Index thread identity lookups and preserve the index across immutable patches. Use the same lookup in command invariants. Keep thread ordering, prior snapshots, and archived/deleted row semantics unchanged.

Verification

  • Parent verification passed all 32 projector, command-invariant, and orchestration-engine tests on current main, including isolated SQLite migrations and command handling.
  • The new 2,000-thread regression checks limited identity reads, immutable prior state, and archived/deleted records.
  • Server package TypeScript checking, targeted lint, formatting, and diff checks passed. Additional configured Effect diagnostics remain a CI gate.
  • This is a pure server lookup change with unchanged event and client contracts. No UI changes, provider calls, or measured end-to-end latency percentage are claimed.

Implemented and verified by gpt-6-astra in T3 Code through the Grok harness.

@vercel

vercel Bot commented Sep 8, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated
akeru-bot-landing Skipped Skipped Sep 8, 2026 7:46pm UTC

Request Review

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:M labels Sep 8, 2026
@greptile-apps

greptile-apps Bot commented Sep 8, 2026

Copy link
Copy Markdown

Greptile Summary

No issues found that require changes before merging. The updated protocol observation behavior, browser attachment flow, and paginated workspace search behavior were exercised successfully, including boundary conditions.

Confidence Score: 5/5

Safe to merge.

No defects were confirmed in the exercised behavior.

Files Needing Attention: None.

T-Rex T-Rex Logs

What T-Rex did

  • Executed focused tests covering disabled defaults, bounded queues, unbounded replay, notification callbacks, and Codex request replies.
  • Ran the focused provider suite covering attachment acquisition, lazy behavior, and runtime headers, plus an eight-case runtime test across multiple connectors and transports; the provider suite passed 93 tests and the eight-case runtime test passed all eight cases, confirming browser metadata and authorization propagation remain correct.
  • Executed paging, deadline-expiry, and cancellation coverage for workspace content search; the focused test passed all three changed-code scenarios and the existing WorkspaceSearchIndex suite passed 11 tests; the update avoids a stale next-page call after the time budget expires while preserving paging and cleanup.
  • Ran the after-command observation workflow and log; the after log recorded the command, working directory, output, and exit code, with 4 files passed and 12 focused tests, confirming the behavior of disabled defaults, bounded/unbounded queues, and Codex callbacks.
  • Restored production baseline and verified clean against HEAD, and removed the temporary authored test file from the source tree.

View all artifacts

T-Rex Ran code and verified through T-Rex

Reviews (1): Last reviewed commit: "Merge remote-tracking branch 'origin/mai..." | Re-trigger Greptile

@leoisadev1
leoisadev1 merged commit bbb5e50 into main Sep 8, 2026
11 checks passed
@leoisadev1
leoisadev1 deleted the perf/server-projector-index branch September 8, 2026 20:39
@github-actions github-actions Bot mentioned this pull request Sep 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant