Skip to content

fix: updated the plan command error message when root cause analysis is not done#162

Closed
MathurAditya724 wants to merge 2 commits intomainfrom
fix/adi/minor-error-message-update
Closed

fix: updated the plan command error message when root cause analysis is not done#162
MathurAditya724 wants to merge 2 commits intomainfrom
fix/adi/minor-error-message-update

Conversation

@MathurAditya724
Copy link
Member

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 4, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

  • (issue) Replace --org/--project flags with /ID syntax by BYK in #161

Bug Fixes 🐛

  • Updated the plan command error message when root cause analysis is not done by MathurAditya724 in #162
  • Added nullable in substatus's zod validation by MathurAditya724 in #157

🤖 This preview updates automatically when you update the PR.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 4, 2026

Codecov Results 📊

✅ Patch coverage is 100.00%. Project has 2022 uncovered lines.
✅ Project coverage is 69.2%. Comparing base (base) to head (head).

Files with missing lines (29)
File Patch % Lines
human.ts 31.87% ⚠️ 682 Missing
resolve-target.ts 10.74% ⚠️ 291 Missing
oauth.ts 25.10% ⚠️ 194 Missing
upgrade.ts 39.76% ⚠️ 153 Missing
api-client.ts 74.21% ⚠️ 138 Missing
resolver.ts 3.23% ⚠️ 120 Missing
errors.ts 5.94% ⚠️ 95 Missing
migration.ts 47.44% ⚠️ 82 Missing
api.ts 89.80% ⚠️ 47 Missing
seer.ts 75.54% ⚠️ 45 Missing
seer.ts 79.87% ⚠️ 30 Missing
preload.ts 39.02% ⚠️ 25 Missing
utils.ts 87.43% ⚠️ 24 Missing
version-check.ts 76.00% ⚠️ 18 Missing
detector.ts 87.79% ⚠️ 16 Missing
schema.ts 64.10% ⚠️ 14 Missing
auth.ts 94.78% ⚠️ 7 Missing
feedback.ts 84.21% ⚠️ 6 Missing
arg-parsing.ts 93.41% ⚠️ 6 Missing
upgrade.ts 93.83% ⚠️ 5 Missing
index.ts 95.06% ⚠️ 4 Missing
colors.ts 91.84% ⚠️ 4 Missing
telemetry.ts 97.20% ⚠️ 4 Missing
env-file.ts 97.17% ⚠️ 3 Missing
sentry-urls.ts 88.00% ⚠️ 3 Missing
alias.ts 98.56% ⚠️ 2 Missing
project-aliases.ts 97.40% ⚠️ 2 Missing
java.ts 97.22% ⚠️ 1 Missing
parser.ts 98.63% ⚠️ 1 Missing
Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    69.20%    69.20%        —%
==========================================
  Files           57        57         —
  Lines         6565      6565         —
  Branches         0         0         —
==========================================
+ Hits          4543      4543         —
- Misses        2022      2022         —
- Partials         0         0         —

Generated by Codecov Action

throw new ValidationError(
"No root causes identified. Cannot create a plan without a root cause."
`Root cause analysis found no causes for issue ${issueId}.\n` +
`Try running 'sentry issue explain ${issueId} --force' to retry the analysis.`
Copy link
Member

Choose a reason for hiding this comment

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

I'd say we should go one step further and offer to run this for the user if this stdin.isTTY is true

@MathurAditya724
Copy link
Member Author

Closing this in favour of #185

@MathurAditya724 MathurAditya724 deleted the fix/adi/minor-error-message-update branch February 5, 2026 18:32
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