Skip to content

Refuse a narration fallback that belongs to another segment - #205

Merged
jmjava merged 1 commit into
mainfrom
cursor/narration-segment-stem
Nov 1, 2026
Merged

jmjava merged 1 commit into
mainfrom
cursor/narration-segment-stem

Conversation

@jmjava

@jmjava jmjava commented Nov 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • Looking up narration for segment 01 also accepted 010-other.md via a broad glob.
  • A stem must be the segment id or start with that id and a hyphen, and a unit test fails if that check is removed.

Test plan

  • tests/test_narrate_from_source.py ignores 010-other.md and still reads 01-demo.md and 01.md
  • complexity and hotspot gates pass against origin/main

Made with Cursor

A glob of 01*.md also matched 010-other.md and treated that file as segment 01.
@jmjava
jmjava merged commit 8ba8add into main Nov 1, 2026
8 checks passed
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