fix(alexandria): point keyless users to sign-in instead of an access error

Draft
JU
6d ago
+5-2

The keyless error for Alexandria commands still says "Alexandria requires a Firecrawl API key with access enabled." That wording dates from the beta (#223). Any Firecrawl API key works now, Free plan included (a few tools need a paid plan or an admin to accept the provider's terms), and the server only refuses keyless callers, so the message sends people looking for an access setting that no longer exists.

The new message says what to do instead:

Alexandria needs a Firecrawl account. Run "firecrawl login --browser" to sign in or create a free one (1,000 credits a month), or set FIRECRAWL_API_KEY.

Changes

  • src/commands/alexandria.ts: new text in requireAlexandriaKey.
  • src/__tests__/alexandria-beta.test.ts: the keyless Find Tools test now checks the message as well as the exit code.

No bundled skill or README repeats the old wording, so nothing else changes.

How I tested it

  • pnpm type-check and pnpm format:check pass.
  • pnpm test: 871 of 872 pass. The one failure (parse.test.ts, "posts to /v2/parse without auth when using the default cloud API with no key") happens on main too when FIRECRAWL_API_KEY is set in the shell, and passes with it unset.
  • firecrawl alexandria find-tools with no key prints the new message and exits 1.

Overlap with #360

#360 edits the same file. This PR changes one string and adds two assertions to an existing test, so it rebases onto #360 trivially, or the other way round.

Open in Capy

Summary by cubic

Fixes the keyless error message for Alexandria commands so users are told how to authenticate instead of being sent looking for an access setting that no longer exists.

The message now points to firecrawl login --browser or setting FIRECRAWL_API_KEY, and the keyless Find Tools test verifies both the exit code and the new message text.