Skip to content

sync upstream main - #101

Merged
tarik02 merged 4 commits into
mainfrom
sync-upstream-main-20260730-2
Jul 30, 2026
Merged

sync upstream main#101
tarik02 merged 4 commits into
mainfrom
sync-upstream-main-20260730-2

Conversation

@tarik02

@tarik02 tarik02 commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

What Changed

  • merged 3 commits from pingdotgg/t3code:main through 9dd425b2234c062b4767583e42d4b2c1aabab15d
  • added conversation-content thread search across server, web, and mobile
  • included the marketing Vercel deployment fix and the upstream user/maintainer documentation split
  • preserved fork-specific goals, settings, and context-menu documentation

Why

Keep the fork aligned with the latest upstream changes while retaining fork-specific behavior.

UI Changes

No fork-specific UI changes. This PR inherits upstream UI changes.

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

Model: gpt-5.6-sol
Harness: T3 Code

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XXL labels Jul 30, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 356acb9879

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

)
)
)
AND messages.text LIKE ${pattern} ESCAPE '!'

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Avoid scanning every message on each search

When a user searches an environment with a large thread history, this leading-wildcard LIKE performs a full scan of projection_thread_messages; the window function must rank the entire matching set before the outer LIMIT 50 can apply. Both clients issue a new query after only 200 ms of typing, and the server uses a synchronous single SQLite connection, so these scans can stall unrelated projection and WebSocket work. Use an indexed search strategy such as FTS, or otherwise actually bound the scanned input rather than only the response.

AGENTS.md reference: AGENTS.md:L17-L19

Useful? React with 👍 / 👎.

import * as NodeFS from "node:fs";

const backups = NodeFS.globSync(
"node_modules/.pnpm/@typescript+native-preview-*/node_modules/@typescript/native-preview-*/lib/tsgo{,.exe}.original*",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Match pnpm's actual native-preview directory names

On the cached pnpm installs this script is intended to repair, the virtual-store path is @typescript+native-preview@<version>/node_modules/@typescript/native-preview/lib/...; the glob instead expects hyphens after both native-preview segments. It therefore matches no backup files, so prepare still reaches effect-tsgo patch with all stale backups present and eventually fails at the documented 101-backup threshold.

Useful? React with 👍 / 👎.

@tarik02
tarik02 merged commit 8dd7c06 into main Jul 30, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XXL 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.

2 participants