Skip to content

Add supported-chains cmd & Improve Error Messages#330

Open
ejacquier wants to merge 4 commits intomainfrom
error-improvement
Open

Add supported-chains cmd & Improve Error Messages#330
ejacquier wants to merge 4 commits intomainfrom
error-improvement

Conversation

@ejacquier
Copy link
Copy Markdown
Contributor

@ejacquier ejacquier commented Mar 30, 2026

Supported Chains

  • Added cre workflow supported-chains command (sorted alphabetically)

📊 Error Handling Improvements

# Top Error (CSV) Count Improvement
1 no project settings file found 321 Now: clearer message ("no CRE project found") + suggests cre init, --project-root, or cd into project
2 Script not found "cre-compile" 303 Now: explains it comes from @chainlink/cre-sdk + suggests bun install / cre update
3 no RPC URLs found 178 Now: shows which target failed + suggests checking rpcs section and --supported-chains
4 failed to parse private key 159 Now: shows actual vs expected length (64 hex chars) + lists common mistakes
5 workflow.yaml: no such file or directory 155 Now: clearer message ("workflow settings file not found") + suggests cre workflow init or checking path

@ejacquier ejacquier requested a review from a team as a code owner March 30, 2026 15:50
@github-actions
Copy link
Copy Markdown

👋 ejacquier, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

@ejacquier ejacquier changed the title Improve Error messages Add supported-chains cmd & Improve Error Messages Mar 30, 2026
Comment thread cmd/workflow/simulate/simulate.go Outdated
target = "(none)"
}
return Inputs{}, fmt.Errorf(
"no RPC URLs found for target %q\n\n"+
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Don't we have a generalised rpc check in the root of the cmd package?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Not quite. cmd/root.go has isLoadDeploymentRPC() which decides whether to load RPC settings (for deploy/pause/activate/delete). That's a
config-loading gate, not a validation error.

Comment thread cmd/workflow/workflow.go Outdated
timothyF95
timothyF95 previously approved these changes Apr 15, 2026
@tarcisiozf tarcisiozf enabled auto-merge April 23, 2026 14:44
@ejacquier ejacquier dismissed timothyF95’s stale review April 23, 2026 16:01

The merge-base changed after approval.

Comment thread cmd/root.go
"cre workflow limits": {},
"cre workflow limits export": {},
"cre workflow build": {},
"cre workflow list": {},
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We're missing cre workflow list during merge conflict resolution

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.

2 participants