Skip to content

Clarify live config and log boundaries#52

Merged
phroi merged 2 commits into
masterfrom
review/live-ops-docs
May 28, 2026
Merged

Clarify live config and log boundaries#52
phroi merged 2 commits into
masterfrom
review/live-ops-docs

Conversation

@phroi
Copy link
Copy Markdown
Member

@phroi phroi commented May 28, 2026

Why

Live testnet operators need the config-generation, preflight funding, and log-boundary details in the public docs without relying on scratch notes or stale WIP.

Changes

  • Document rebuilding ignored live configs from environment variables and the optional testnet RPC URL.
  • Clarify preflight funding checks, balance fields, and direct machine-readable JSON usage.
  • Distinguish dynamic validation sessions from production bot-only logs and note the tester retry-budget boundary.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the documentation across several README files to explain how to rebuild disposable live configurations from environment variables, retrieve funding addresses, and manage production bot-only logs. The review feedback points out a broken relative link to another README file in apps/supervisor/README.md and suggests formatting it as a proper relative markdown link.

Comment thread apps/supervisor/README.md Outdated
@phroi
Copy link
Copy Markdown
Member Author

phroi commented May 28, 2026

/gemini review

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the documentation in README.md, apps/supervisor/README.md, and apps/tester/README.md to explain how to rebuild disposable live configurations from environment variables using pnpm live:config-from-env -- --force. It also details how to retrieve public funding addresses, run preflight checks, output machine-readable JSON, and clarifies the directory structure for production bot-only logs. There are no review comments, and I have no feedback to provide.

@phroi phroi merged commit 80990fb into master May 28, 2026
1 check passed
@phroi
Copy link
Copy Markdown
Member Author

phroi commented May 28, 2026

LGTM

Phroi %22

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.

1 participant