Skip to content

Add blocks project seed for seeding - #1002

Merged
jamdelion merged 4 commits into
mainfrom
jh/blocks-project-seeds
Sep 8, 2026
Merged

Add blocks project seed for seeding#1002
jamdelion merged 4 commits into
mainfrom
jh/blocks-project-seeds

Conversation

@jamdelion

Copy link
Copy Markdown
Contributor

What's changed?

PR description written by Claude, and reviewed by JH

Adds a Blocks (Scratch) starter project to the seed data, so developers working on integrated
Blocks editor projects in projects-ui have a real example locally instead of having to hand-craft
a scratch_component row.

  • New lib/tasks/project_components/neil-the-seal-starter/ containing neil-the-seal-starter.sb3
    and a project_config.yml with TYPE: "code_editor_scratch". Picked up automatically by
    rails projects:create_all, which db/seeds.rb already invokes.
  • The .sb3 is copied unmodified from
    raspberrypilearning/editor-neil-the-seal
    at en/code/neil-the-seal-starter/neil-the-seal-starter.sb3. The identifier
    neil-the-seal-starter matches production, so local and deployed data agree.
  • Documents Scratch seeds in lib/tasks/README.md, which previously only covered Python and HTML.
    Most usefully it warns that content repositories use a lowercase-keyed project_config.yml
    read by the webhook/UploadJob path, whereas the seeding task reads uppercase keys — copying
    one across verbatim parses fine but silently produces a nameless Python project.
  • Relaxes the README's claim that IDENTIFIER is "three words separated by dashes"; it isn't
    enforced and existing projects such as editor-scratch-testing-starter already use more.

Related

Reaching this project through projects-ui also needs the projects-admin seed that sets
direct_to_editor and editor_starter_project — this happens using the production-to-local script. Once both are
seeded it's at /en/projects/editor-neil-the-seal/editor. It's unlisted and in no pathway,
matching the current status of production, so it can't be reached by browsing.

Copilot AI lite review requested due to automatic review settings September 7, 2026 15:23
@cla-bot

cla-bot Bot commented Sep 7, 2026

Copy link
Copy Markdown

We require contributors to sign our Contributor License Agreement, and we don't have you on file. In order for us to review and merge your code, please complete this form and we'll get you added and review your contribution as soon as possible.

Copilot AI 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.

🟡 Changes recommended

Newly added README guidance contains a couple of misleading/unclear statements that should be corrected to avoid developer confusion when running seeds.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Adds a Scratch (Blocks) starter project into the local seed/import pipeline so developers can work against realistic Scratch project data, and updates the seeding documentation to cover Scratch projects.

Changes:

  • Add a new Scratch starter seed directory for neil-the-seal-starter (config + .sb3).
  • Document how Scratch seeds are imported (and how this differs from content-repo project_config.yml files).
  • Rename the Hello World starter project identifier to editor-hello-world-starter.
File summaries
File Description
lib/tasks/README.md Documents Scratch seed structure and config conventions, plus identifier guidance.
lib/tasks/project_components/neil-the-seal-starter/project_config.yml Adds seed metadata for the new Scratch starter project.
lib/tasks/project_components/neil-the-seal-starter/neil-the-seal-starter.sb3 Adds the Scratch starter project archive used by the importer.
lib/tasks/project_components/hello_world_starter/project_config.yml Updates the seeded project identifier for Hello World.
Review details
  • Files reviewed: 3/4 changed files
  • Comments generated: 2
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread lib/tasks/README.md Outdated
Comment thread lib/tasks/README.md Outdated
@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown

Test coverage

93.57% line coverage reported by SimpleCov.
Run: https://github.com/RaspberryPiFoundation/editor-api/actions/runs/34213346003

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@cla-bot

cla-bot Bot commented Sep 7, 2026

Copy link
Copy Markdown

We require contributors to sign our Contributor License Agreement, and we don't have you on file. In order for us to review and merge your code, please complete this form and we'll get you added and review your contribution as soon as possible.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@cla-bot

cla-bot Bot commented Sep 7, 2026

Copy link
Copy Markdown

We require contributors to sign our Contributor License Agreement, and we don't have you on file. In order for us to review and merge your code, please complete this form and we'll get you added and review your contribution as soon as possible.

@zetter-rpf zetter-rpf 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.

Nice one.

I've added you to the CLA list so you should be able to merge once you rebase/merge in main.

@cla-bot cla-bot Bot added the cla-signed label Sep 8, 2026
@jamdelion
jamdelion enabled auto-merge (squash) September 8, 2026 10:04
@jamdelion
jamdelion merged commit b4f1bab into main Sep 8, 2026
8 checks passed
@jamdelion
jamdelion deleted the jh/blocks-project-seeds branch September 8, 2026 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants