Skip to content

Refactor: Bump clack version to latest and small fixes#264

Merged
kfirstri merged 3 commits intomainfrom
bump-clack
Feb 18, 2026
Merged

Refactor: Bump clack version to latest and small fixes#264
kfirstri merged 3 commits intomainfrom
bump-clack

Conversation

@kfirstri
Copy link
Collaborator

@kfirstri kfirstri commented Feb 18, 2026

Note

Description

This PR upgrades the @clack/prompts package from v0.11.0 to v1.0.1 and refactors error handling to use the new API methods. The upgrade includes improved visual feedback for errors and canceled operations using spinner.error() and spinner.cancel() instead of generic spinner.stop() calls.

Related Issue

None

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Other (please describe):

Changes Made

  • Upgraded @clack/prompts from ^0.11.0 to ^1.0.1 (includes @clack/core bump)
  • Updated OAuth connector flow to use spinner.cancel() when user skips authorization
  • Changed error handling in runTask() to use spinner.error() instead of spinner.stop() for failures
  • Fixed OAuth flow error states to properly use spinner.error() for failed/timed out authorizations
  • Removed unnecessary newline in OAuth prompt log message
  • Updated code comment to mention both Ctrl+C and Esc for skipping
  • Removed redundant TypeScript narrowing comment in OAuth flow

Testing

  • I have tested these changes locally
  • I have added/updated tests as needed
  • All tests pass (npm test)

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (if applicable)
  • My changes generate no new warnings
  • I have updated docs/ (AGENTS.md) if I made architectural changes

Additional Notes

The new clack v1 API provides better semantic methods for spinner states:

  • spinner.cancel() - for user-initiated cancellations (Esc/Ctrl+C)
  • spinner.error() - for failures and error states
  • spinner.stop() - for successful completions

This improves UX by providing clearer visual feedback to users about different operation outcomes.


🤖 Generated by Claude | 2026-02-18 12:34 UTC

@kfirstri kfirstri self-assigned this Feb 18, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 18, 2026

🚀 Package Preview Available!


Install this PR's preview build with npm:

npm i @base44-preview/cli@0.0.32-pr.264.aa4c901

Prefer not to change any import paths? Install using npm alias so your code still imports base44:

npm i "base44@npm:@base44-preview/cli@0.0.32-pr.264.aa4c901"

Or add it to your package.json dependencies:

{
  "dependencies": {
    "base44": "npm:@base44-preview/cli@0.0.32-pr.264.aa4c901"
  }
}

Preview published to npm registry — try new features instantly!

@kfirstri kfirstri requested a review from Paveltarno February 18, 2026 12:33
@claude
Copy link

claude bot commented Feb 18, 2026

README check ran. 4 commands were missing from the table and 1 description was updated. Added: eject, connectors pull, connectors push, types generate. Updated deploy description to match current implementation. README.md has been updated in this branch.

Copy link
Collaborator

@Paveltarno Paveltarno left a comment

Choose a reason for hiding this comment

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

NICE!

@kfirstri kfirstri merged commit 2689385 into main Feb 18, 2026
11 checks passed
@kfirstri kfirstri deleted the bump-clack branch February 18, 2026 13:16
@kfirstri kfirstri mentioned this pull request Feb 18, 2026
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

Comments