Skip to content

MAINT: Fix demo scripts and missing files.#894

Open
lenabadinter wants to merge 1 commit intotraceloop:mainfrom
lenabadinter:scratch/lena.badinter/fix-missing-scripts-files
Open

MAINT: Fix demo scripts and missing files.#894
lenabadinter wants to merge 1 commit intotraceloop:mainfrom
lenabadinter:scratch/lena.badinter/fix-missing-scripts-files

Conversation

@lenabadinter
Copy link

@lenabadinter lenabadinter commented Mar 8, 2026

Remove scripts that were not working because the required files were missing.

Summary by CodeRabbit

  • Chores
    • Removed several npm scripts from the development environment, including image generation and MCP-related commands.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


lena.badinter seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 8, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 316da016-1116-46f5-8ca2-ca073eac36b0

📥 Commits

Reviewing files that changed from the base of the PR and between 9df89a0 and 4e31700.

📒 Files selected for processing (1)
  • packages/sample-app/package.json
💤 Files with no reviewable changes (1)
  • packages/sample-app/package.json

📝 Walkthrough

Walkthrough

Six npm scripts were removed from the sample-app package.json file: run:image_generation, run:sample_edit, run:sample_generate, run:mcp:real, run:mcp:working, and dev:image_generation. No scripts were added.

Changes

Cohort / File(s) Summary
Script Cleanup
packages/sample-app/package.json
Removed six npm scripts from the scripts section. No new scripts added. This reduces the available command set for the sample-app package.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A rabbit hops through scripts with glee,
Six commands bundled in a spree,
But simplicity wins the day—
Unused paths now fade away! 🐰
The package.json dances light,
Cleaner code, a rabbit's delight.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'MAINT: Fix demo scripts and missing files' is related to the changeset, which removes demo scripts, but is partially misleading as it suggests files are being fixed when they are actually being removed.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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