Skip to content

ref: Rename config folder from .sentry-cli-next to .sentry#50

Merged
BYK merged 2 commits intomainfrom
byk/rename-config-dir
Jan 23, 2026
Merged

ref: Rename config folder from .sentry-cli-next to .sentry#50
BYK merged 2 commits intomainfrom
byk/rename-config-dir

Conversation

@BYK
Copy link
Copy Markdown
Member

@BYK BYK commented Jan 23, 2026

Summary

Renames the config folder from ~/.sentry-cli-next to ~/.sentry and updates the environment variable from SENTRY_CLI_CONFIG_DIR to SENTRY_CONFIG_DIR.

Changes

  • Update default config directory path in src/lib/config.ts and src/context.ts
  • Rename environment variable for custom config directory
  • Update JSDoc comments in DSN cache/types modules
  • Update README documentation
  • Update all test files to use the new environment variable name

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 23, 2026

Codecov Results 📊

✅ Patch coverage is 100.00%. Project has 1179 uncovered lines.
❌ Project coverage is 58.34%. Comparing base (base) to head (head).

Files with missing lines (18)
File Patch % Lines
human.ts 16.37% ⚠️ 378 Missing
oauth.ts 21.71% ⚠️ 202 Missing
api-client.ts 49.77% ⚠️ 109 Missing
resolver.ts 3.77% ⚠️ 102 Missing
errors.ts 5.94% ⚠️ 95 Missing
fixture.ts 28.74% ⚠️ 62 Missing
api.ts 66.30% ⚠️ 61 Missing
errors.ts 51.61% ⚠️ 60 Missing
config.ts 83.73% ⚠️ 34 Missing
preload.ts 39.02% ⚠️ 25 Missing
detector.ts 87.79% ⚠️ 16 Missing
cache.ts 76.27% ⚠️ 14 Missing
colors.ts 85.42% ⚠️ 7 Missing
index.ts 95.06% ⚠️ 4 Missing
parser.ts 93.75% ⚠️ 4 Missing
env-file.ts 97.17% ⚠️ 3 Missing
alias.ts 96.43% ⚠️ 2 Missing
java.ts 97.22% ⚠️ 1 Missing
Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
- Coverage    64.18%    58.34%    -5.84%
==========================================
  Files           28        33        +5
  Lines         2211      2830      +619
  Branches         0         0         —
==========================================
+ Hits          1419      1651      +232
- Misses         792      1179      +387
- Partials         0         0         —

Generated by Codecov Action

- Update default config directory from ~/.sentry-cli-next to ~/.sentry
- Rename environment variable from SENTRY_CLI_CONFIG_DIR to SENTRY_CONFIG_DIR
- Export constants CONFIG_DIR_ENV_VAR and DEFAULT_CONFIG_DIR_NAME for reuse
- Export getConfigDir() helper and use it in context.ts
- Update test files to use the exported constant
@BYK BYK force-pushed the byk/rename-config-dir branch from 0e7caf0 to 8d17efb Compare January 23, 2026 13:20
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 23, 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) Add short ID aliases for multi-project support by BYK in #31

Internal Changes 🔧

  • (release) Fix changelog-preview permissions by BYK in #41
  • Rename config folder from .sentry-cli-next to .sentry by BYK in #50

🤖 This preview updates automatically when you update the PR.

- Add test/helpers.ts with createTestConfigDir() and cleanupTestDir()
- Use fs.promises.mkdtemp with os.tmpdir() for proper temp directory isolation
- Remove hardcoded /tmp fallback that wouldn't work on Windows
- Simplify beforeEach/afterEach in E2E and lib tests
@BYK BYK marked this pull request as ready for review January 23, 2026 13:32
@BYK BYK enabled auto-merge (squash) January 23, 2026 13:33
@BYK BYK merged commit 5146c36 into main Jan 23, 2026
15 checks passed
@BYK BYK deleted the byk/rename-config-dir branch January 23, 2026 13:34
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