sync upstream main - #99
Conversation
Co-authored-by: Julius Marminge <julius0216@outlook.com> Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…gdotgg#2679) Co-authored-by: codex <codex@users.noreply.github.com>
…e diff HEAD --numstat (pingdotgg#4843)
…otgg#4853) Co-authored-by: Simon Doba <simon.doba@orbit.de> Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
…gdotgg#4905) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
pingdotgg#4897) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…ngdotgg#4882) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…it (pingdotgg#4949) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…ingdotgg#4867) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…gdotgg#4964) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…oring (pingdotgg#4967) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: fd2b9d8643
ℹ️ 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".
| const refreshedTelemetry = telemetry.refresh.pipe(Effect.catch(() => telemetry.latest)); | ||
| const read: ProcessDiagnostics["Service"]["read"] = refreshedTelemetry.pipe( |
There was a problem hiding this comment.
Bundle the resource monitor with the Nix server
For installations built through the fork's supported nix/package.nix, the build only produces the web/server JavaScript bundles and the install phase copies only those outputs; it never builds or installs t3-resource-monitor. Consequently telemetry.refresh always fails in that environment, and this fallback returns the initial empty telemetry snapshot, so Diagnostics no longer lists or signals provider, terminal, or other server child processes after the previous ps-based implementation was removed. Build and copy the monitor in the Nix derivation, or retain a process-query fallback when the sidecar is unavailable.
AGENTS.md reference: AGENTS.md:L21-L23
Useful? React with 👍 / 👎.
What Changed
pingdotgg/t3code:mainthrough197d3487116372ca2f16661f57d4e869fc3cb6c0Why
Keep the fork aligned with the latest upstream changes.
UI Changes
No fork-specific UI changes. This PR inherits upstream UI changes.
Checklist
Model: gpt-5.6-sol
Harness: Codex