Fix activity completion and progression issues in activities 30-37

Browse source

This commit addresses two critical issues:

1. Completion Bug (activities 30-37):
   - Final steps were looping forever, preventing activity completion
   - Fixed by removing next_section_and_step from completion transitions
   - Kept off_topic transition looping to avoid validator terminal step errors
   - Activities now complete properly when users give valid final answers

2. Activity37 Bucket Logic:
   - Changed "close" bucket to retry same step instead of advancing
   - Only "correct" bucket now advances to next step
   - All other buckets (close, incomplete, wrong_language, etc.) retry
   - This ensures students must get correct answers to progress

Technical Details:
- Final steps are not considered "terminal" if at least one transition
  has next_section_and_step (validator requirement)
- Off-topic transitions loop back to allow another attempt
- Completion happens when get_next_step() returns None, None

Validation:
- All 8 activities pass activity_yaml_validator.py
- No errors or warnings

Affects: activity30-37 (all new merged activities)

...

This commit is contained in:

Claude 2025-11-08 22:59:06 +00:00

parent 08bbc47a6b

commit f774d36d74

No known key found for this signature in database

8 changed files with 6612 additions and 5697 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

1083

research/activity30-logic-puzzles.yaml

View file

File diff suppressed because it is too large Load diff

1403

research/activity31-scientific-method.yaml

View file

File diff suppressed because it is too large Load diff

1644

research/activity32-world-geography.yaml

View file

File diff suppressed because it is too large Load diff

1317

research/activity33-environmental-science.yaml

View file

File diff suppressed because it is too large Load diff

1471

research/activity34-media-literacy.yaml

View file

File diff suppressed because it is too large Load diff

1479

research/activity35-american-history.yaml

View file

File diff suppressed because it is too large Load diff

1906

research/activity36-biblical-history.yaml

View file

File diff suppressed because it is too large Load diff

2006

research/activity37-programming-languages.yaml

View file

File diff suppressed because it is too large Load diff