Skip to content

[DIT-13305] Show error message when a scan exceeds plan limits. Make path param for scan optional.#143

Merged
jholiga merged 1 commit into
masterfrom
joey/dit-13305-improve-ditto-scan-installation-flow
Jul 21, 2026
Merged

[DIT-13305] Show error message when a scan exceeds plan limits. Make path param for scan optional.#143
jholiga merged 1 commit into
masterfrom
joey/dit-13305-improve-ditto-scan-installation-flow

Conversation

@jholiga

@jholiga jholiga commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Overview

This PR introduces a couple small changes so that the CLI and specifically ditto scan are more productionized:

  1. Show a formatted error message when a scan is run on a free or trial plan and exceeds the limit of extracted candidate strings (1000)
  2. Make the path param for the scan optional. Omitting the path defaults to the cwd. This allows a user to run npx @dittowords/cli scan which will install, prompt the user for an API token, and run the scan from the current directory, all in one command.

Context

Goes hand-in-hand with PR: here

Screenshots

image

Test Plan

Testing successfully completed in via:

  • Ensure unit tests pass
  • Test the CLI works on a repo without passing the path argument, e.g. yarn scan from the cli repo will run the scan on the CLI repo.
  • Run the ditto scan on a repo that trips the max candidates error and ensure the error displays properly with a link to upgrade plan

@jholiga
jholiga requested a review from joustrich July 21, 2026 00:25
@jholiga
jholiga marked this pull request as ready for review July 21, 2026 00:25
@jholiga

jholiga commented Jul 21, 2026

Copy link
Copy Markdown
Contributor Author

Going to merge but hold off on a full CLI package release, @tomkowalski will be doing a fast followup of this work that builds on the trial scan string limit functionality.

@jholiga
jholiga merged commit 78835f1 into master Jul 21, 2026
1 check passed
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