Skip to content

fix(init): remove implementation detail from help text#385

Merged
betegon merged 1 commit intomainfrom
fix/init-help-text
Mar 10, 2026
Merged

fix(init): remove implementation detail from help text#385
betegon merged 1 commit intomainfrom
fix/init-help-text

Conversation

@betegon
Copy link
Member

@betegon betegon commented Mar 10, 2026

Summary

Cleans up the sentry init help text per @BYK's review feedback:

  • Removes the sentence about remote workflow coordination (implementation detail, not useful to users)
  • Broadens "configure error monitoring" to "configure Sentry" since the wizard sets up tracing, replays, crons, etc. — not just errors

Test Plan

  • bun run typecheck passes
  • bun test test/commands/init.test.ts passes (7/7)

Drop the sentence about remote workflow coordination from fullDescription
and broaden "configure error monitoring" to "configure Sentry" since the
wizard sets up more than just error monitoring.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Contributor

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 ✨

Init

  • Add git safety checks before wizard modifies files by betegon in #379
  • Add experimental warning before wizard runs by betegon in #378
  • Add init command for guided Sentry project setup by betegon in #283

Other

  • (issue-list) Redesign table to match Sentry web UI by BYK in #372
  • Return-based output with OutputConfig on buildCommand by BYK in #380
  • Add --fields flag for context-window-friendly JSON output by BYK in #373
  • Magic @ selectors (@latest, @most_frequent) for issue commands by BYK in #371
  • Input hardening against agent hallucinations by BYK in #370
  • Add response caching for read-only API calls by BYK in #330

Bug Fixes 🐛

Init

  • Remove implementation detail from help text by betegon in #385
  • Truncate uncommitted file list to first 5 entries by MathurAditya724 in #381

Other

  • (docs) Remove double borders and fix column alignment on landing page tables by betegon in #369
  • Add trace ID validation to trace view + UUID dash-stripping by BYK in #375

Internal Changes 🔧

Init

  • Remove --force flag by betegon in #377
  • Remove dead determine-pm step label by betegon in #374

Other

  • Converge Tier 1 commands to writeOutput helper by BYK in #376

🤖 This preview updates automatically when you update the PR.

@github-actions
Copy link
Contributor

Codecov Results 📊

104 passed | Total: 104 | Pass Rate: 100% | Execution Time: 0ms

📊 Comparison with Base Branch

Metric Change
Total Tests
Passed Tests
Failed Tests
Skipped Tests

✨ No test changes detected

All tests are passing successfully.

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

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    95.48%    95.49%    +0.01%
==========================================
  Files          142       142         —
  Lines        19898     19898         —
  Branches         0         0         —
==========================================
+ Hits         18999     19000        +1
- Misses         899       898        -1
- Partials         0         0         —

Generated by Codecov Action

@betegon betegon marked this pull request as ready for review March 10, 2026 18:11
@betegon betegon merged commit 6baae83 into main Mar 10, 2026
21 checks passed
@betegon betegon deleted the fix/init-help-text branch March 10, 2026 18:11
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