Skip to content

Scope Prompt A to what rules.yaml actually checks - #14

Merged
alex-16moro merged 1 commit into
mainfrom
cursor/prompt-registry-scope-3eed
Sep 20, 2026
Merged

alex-16moro merged 1 commit into
mainfrom
cursor/prompt-registry-scope-3eed

Conversation

@alex-16moro

@alex-16moro alex-16moro commented Sep 20, 2026 •

Copy link
Copy Markdown
Owner

Summary

Prompt A was still sending the first engineer into work the registry does not check: full Euler/DDPM source, docs search, catch-early, “0 findings,” and implied public-API extras.

The paste and /scaffold now:

  • Copy the two overlay templates (scheduling_heun_lite.py + matching test)
  • Ground in conventions/rules.yaml blocking ids only
  • Stop at file-scoped convention_check.py 0 blocking
  • Keep TODO(engineer) in step() — skips without torch are success

They no longer ask the engineer to read scheduling_euler_discrete.py / scheduling_ddpm.py, run docs_mcp_server.py --query, or chase exports/docs/sampler math.

Fork overlay copy: alex-16moro/diffusers#10

Convention gate

Docs / overlay command / generated first-task text. No scheduler files.

Tests run

  • python3 -m unittest tests.test_tooling.TestEngineerPromptScope tests.test_tooling.TestAttachEmptyMcp tests.test_tooling.TestProjectionDrift tests.test_tooling.TestOverlayPrGate tests.test_tooling.TestDemoContribute -v — 10 passed
  • make test — 52 passed, 8 skipped (no torch)
  • make drift — clean

make check (convention_check.py --all on the kit) is still red on tests/schedulers/test_scheduling_euler_lite.py TEST002 dtype. That file landed on main before this PR; this change does not touch it.

Open in Web Open in Cursor 

Prompt A and /scaffold now copy the overlay templates and stop at a
file-scoped 0-blocking gate. They no longer send the engineer into
Euler/DDPM source, docs search, catch-early, or public-API extras the
registry does not scan.

Co-authored-by: ale93.moro <ale93.moro@gmail.com>
@alex-16moro
alex-16moro marked this pull request as ready for review September 20, 2026 10:51
@alex-16moro
alex-16moro merged commit 3a02376 into main Sep 20, 2026
1 check failed
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