Repository navigation
feat(platform): add minimal runtime logs - #683
Conversation
The test runs two real Vite builds, which take about four seconds and come close to the default five-second timeout under load. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Add `--runtime-logs full|minimal` to platform install and upgrade. In minimal mode, platform and project Workers run with Workers Logs off, the tail Worker stores no URL, log message or exception message, and dispatch metering drops the request path. Outcome, method, status and deployment id are still recorded. The setting is saved in the deployment manifest (schema version 5) and existing project Workers are updated after each upgrade or repair while minimal. Fixes #667 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
🚀 Deploying Preview to Cloudflare 🚀Preview URL: https://feat-platform-minimal-telemetry-void-cloud.voidzero-docs.workers.dev (commit 14fafb0)This URL reflects your latest Preview deploymentPreview Deployments by commit
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: fc872f81d1
ℹ️ 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".
…ell minimal rows apart A Durable Object restore now always applies the installation's current Workers Logs setting, so a release saved in minimal mode cannot turn logs off after a switch back to full. Minimal log rows store no URL, so two requests in the same millisecond could look identical and be merged by the readers. Minimal rows now keep the request Ray ID in blob8, and the readers group by it. Co-Authored-By: Claude Opus 5.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: 32398d37d5
ℹ️ 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".
The live tail dropped any row it had already shown by shape. With minimal runtime logs, two requests in the same millisecond can share a shape, so the tail now counts each shape and shows only rows beyond the count it has already seen. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Fixes #667
Adds an opt-in setting that keeps request URLs, log messages and exception messages out of platform and project Worker telemetry. The default stays
full, so nothing changes unless you ask for it.void platform install --runtime-logs minimal void platform upgrade [id] --runtime-logs minimal # or full, to switch backWhat
minimalchangesHow the setting flows
Answers to the issue's questions
--runtime-logs full|minimalflag on install and upgrade, saved in the manifest. Later upgrades keep it.Guards
runtimeLogs: 1in the runtime manifest). Install, upgrade and rollback refuse a runtime that does not.repairkeeps the saved setting. Onlyupgradecan change it.Runtime logs: minimal (was full)when the value changes.Known gaps (left as is)
upgrade/repairsweep fixes it.Tests
A second commit raises the timeout of a Container build test that runs two real Vite builds (about 4s against the 5s default).
🤖 Generated with Claude Code