Skip to content

chore(sync): merge pingdotgg/t3code main - #315

Merged
Yarden-zamir merged 25 commits into
mainfrom
fork-sync/2026-10-23-1228
Oct 23, 2026
Merged

Yarden-zamir merged 25 commits into
mainfrom
fork-sync/2026-10-23-1228

Conversation

@Yarden-zamir

@Yarden-zamir Yarden-zamir commented Oct 23, 2026 •

Copy link
Copy Markdown
Owner

Fork Sync failed on merge conflicts: https://github.com/Yarden-zamir/t3code/actions/runs/39943621573. Issue: #286.

Conflicts

Adopted upstream pull requests

All 7 (pingdotgg#16653 to pingdotgg#16719) are unchanged at their listed heads. Migration check: 68 migrations, all in upstream main. Adopted list check: passes.

Upstream changes in fork areas

The branch also merges origin/main (#296, #312), because main moved during the repair.

Checks

  • checks.py of the fork-repair skill: typecheck (35 of 35), migration check and adopted pull request check pass.
  • Fork tests on the VPS: 98 files outside apps/server and apps/web, 42 apps/web files and 28 of 82 apps/server files pass. The other 54 server files did not run here: a test run in that range killed the agent session twice (the live T3 service did not restart). Fork CI runs all of them.

t3dotgg and others added 24 commits October 23, 2026 03:08
…g#24063)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…t started them (pingdotgg#21299)

Co-authored-by: Julius Marminge <julius0216@outlook.com>
Co-authored-by: Theo Browne <me@theo.li>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…matches (pingdotgg#24032)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: PR Batch Tester <agent@local.test>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…ngdotgg#24008)

Co-authored-by: fixfon <12512757+fixfon@users.noreply.github.com>
Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
…uring agent screenshots (pingdotgg#24056)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: PR Batch Tester <agent@local.test>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…otgg#23873)

Co-authored-by: PR Batch Tester <agent@local.test>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…29s (pingdotgg#23889)

Co-authored-by: PR Batch Tester <agent@local.test>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…or finishes (pingdotgg#23752)

Co-authored-by: PR Batch Tester <agent@local.test>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
# Conflicts:
#	apps/desktop/src/preview/DesktopBrowserHost.ts
#	apps/server/src/mcp/OrchestratorMcpService.ts
#	apps/server/src/mcp/toolkits/orchestrator/tools.test.ts
#	apps/web/src/components/chat/MessagesTimeline.tsx
#	apps/web/src/components/sidebar/SidebarThreadRow.tsx
Upstream pingdotgg#23991 restores ScheduledTaskService.interval.test.ts. The fork scheduled task service reads server settings, so its test layer needs ServerSettings.layerTest().
…1228

# Conflicts:
#	apps/server/src/mcp/toolkits/orchestrator/tools.test.ts
Upstream pingdotgg#23983 fails new polling loops in tests. RemoteDelegation.test.ts comes from adopted pull request pingdotgg#16719, which the fork keeps as it is, so the rule stays at warn for that file. The fork rule of the entry names the override, so that it goes when the loops are converted.
@coderabbitai

coderabbitai Bot commented Oct 23, 2026

Copy link
Copy Markdown

Review in Change Stack →

✨ Finishing Touches: 💡 1
🛠️ Fix failing CI checks: 💡
  • Commit to this branch
  • Create a new PR
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@Yarden-zamir
Yarden-zamir merged commit c806985 into main Oct 23, 2026
18 of 19 checks passed
@Yarden-zamir
Yarden-zamir deleted the fork-sync/2026-10-23-1228 branch October 23, 2026 14:41
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.

7 participants