Skip to content

Conversation

@jribbink
Copy link
Contributor

@jribbink jribbink commented Jan 26, 2026

Closes #1649

Related: onflow/flow-cli#2254

@vercel
Copy link

vercel bot commented Jan 26, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Jan 31, 2026 0:20am

Request Review

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds documentation for the new flow transactions profile command, which enables developers to profile computational performance of sealed transactions on any Flow network.

Changes:

  • Add comprehensive documentation page for the flow transactions profile command
  • Update CLI index to mention transaction profiling capabilities
  • Add examples and reference to transaction profiling in commands documentation
  • Integrate transaction profiling into the Cadence computation profiling guide

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
docs/build/tools/flow-cli/transactions/profile-transactions.md New comprehensive documentation page covering usage, analysis methods, requirements, and implementation details for transaction profiling
docs/build/tools/flow-cli/index.md Added transaction profiling to the list of CLI capabilities
docs/build/tools/flow-cli/commands.md Added transaction profiling examples and documentation link to the commands reference
docs/build/cadence/advanced-concepts/computation-profiling.md Updated computation profiling guide to include transaction profiling as one of three complementary approaches

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +185 to +192
### Output Format

- Flag: `--output`
- Short Flag: `-o`
- Valid inputs: `json`, `inline`

Specify the format of the command results displayed in the console.

Copy link

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

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

The --output flag is documented twice with conflicting descriptions. Lines 155-160 describe it as setting the profile output file path with default profile-{tx_id_prefix}.pb.gz, while lines 187-191 describe it as setting the output format (json, inline). The second definition appears to be incorrect for this command and should be removed or corrected.

Suggested change
### Output Format
- Flag: `--output`
- Short Flag: `-o`
- Valid inputs: `json`, `inline`
Specify the format of the command results displayed in the console.

Copilot uses AI. Check for mistakes.

The Flow CLI provides a command to profile the computational performance of sealed transactions on any Flow network. This diagnostic tool generates detailed CPU profiles in the industry-standard `pprof` format, allowing you to analyze exactly where computation is being spent during transaction execution.

The command works by forking the blockchain state and replaying the transaction in an isolated environment, ensuring accurate profiling results that match the original execution.
Copy link
Member

Choose a reason for hiding this comment

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

Can you link to forking here or put an info bubble like "To read more about forking..."

@jribbink jribbink enabled auto-merge (squash) January 31, 2026 00:20
@jribbink jribbink merged commit fef1fb9 into main Jan 31, 2026
3 checks passed
@jribbink jribbink deleted the jribbink/profile-docs branch January 31, 2026 00:20
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.

Create flow transactions profile command docs

3 participants