Skip to content

Remove beta-skills folder - #271

Merged
developersdigest merged 2 commits into
mainfrom
remove-beta-skills
Sep 21, 2026
Merged

developersdigest merged 2 commits into
mainfrom
remove-beta-skills

Conversation

@developersdigest

@developersdigest developersdigest commented Sep 21, 2026 •

Copy link
Copy Markdown
Member

Remove the beta-skills folder and its npm package entry. No ID behavior or setup implementation changes.

The existing setup alexandria command still references the removed folder; this PR intentionally contains only the requested bundle removal.

Validation: git diff --check. Deletion-only cleanup plus package file-list update; no runtime tests rerun.


Summary by cubic

Removes the obsolete beta-skills bundle and packages the stable skills directory instead. Alexandria setup now installs firecrawl-alexandria and firecrawl-agent from the stable bundle; request IDs, scrape IDs, and their output behavior are unchanged.

Written for commit 5d7def9. Summary will update on new commits.

Review in cubic

@developersdigest developersdigest changed the title Replace beta skill bundle with stable skills Remove beta-skills folder Sep 21, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

1 issue found across 4 files

Confidence score: 2/5

  • package.json's firecrawl setup alexandria handler passes ../../beta-skills to npx, but published packages no longer include that directory, so the setup command fails for users; remove or deprecate the handler, or repoint it to a shipped location.
Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="package.json">

<violation number="1" location="package.json:74">
P1: `firecrawl setup alexandria` now fails for published packages because its handler still passes `../../beta-skills` to `npx`, but this package no longer ships that directory. Remove or deprecate that handler, or repoint it to an available skill source before removing the package entry.</violation>
</file>

Shadow auto-approve: would not auto-approve because issues were found.

Fix all with cubic | Re-trigger cubic

Comment thread package.json
},
"files": [
"dist",
"beta-skills",

@cubic-dev-ai cubic-dev-ai Bot Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P1: firecrawl setup alexandria now fails for published packages because its handler still passes ../../beta-skills to npx, but this package no longer ships that directory. Remove or deprecate that handler, or repoint it to an available skill source before removing the package entry.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At package.json, line 74:

<comment>`firecrawl setup alexandria` now fails for published packages because its handler still passes `../../beta-skills` to `npx`, but this package no longer ships that directory. Remove or deprecate that handler, or repoint it to an available skill source before removing the package entry.</comment>

<file context>
@@ -71,7 +71,6 @@
   "files": [
     "dist",
-    "beta-skills",
     "README.md"
   ],
   "packageManager": "pnpm@10.12.1",
</file context>
Fix with cubic

@developersdigest
developersdigest merged commit 1fa031a into main Sep 21, 2026
8 checks passed
@developersdigest
developersdigest deleted the remove-beta-skills branch September 21, 2026 23:58
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