Skip to content

Update notes and fix scene lifestyle issues - #79

Merged
deanblackborough merged 1 commit into
mainfrom
release-fixes-multi-select
Jun 16, 2026
Merged

Update notes and fix scene lifestyle issues#79
deanblackborough merged 1 commit into
mainfrom
release-fixes-multi-select

Conversation

@deanblackborough

@deanblackborough deanblackborough commented Jun 16, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Bug Fixes
    • Corrected scene initialisation lifecycle to prevent loaded content from being overwritten during scene activation, ensuring more reliable data persistence.

@deanblackborough
deanblackborough merged commit b9f7086 into main Jun 16, 2026
1 of 2 checks passed
@deanblackborough
deanblackborough deleted the release-fixes-multi-select branch June 16, 2026 21:45
@coderabbitai

coderabbitai Bot commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: ce1e57ca-25a0-46ff-95fc-be8c430c43d5

📥 Commits

Reviewing files that changed from the base of the PR and between dc33d7e and 1618115.

📒 Files selected for processing (7)
  • NOTES.md
  • README.md
  • src/prune/app/app.cpp
  • src/prune/scene/artillery/artillery_scene.cpp
  • src/prune/scene/platformer/platformer_scene.cpp
  • src/prune/scene/scene.hpp
  • src/prune/scene/simple_shooter/simple_shooter_scene.cpp

📝 Walkthrough

Walkthrough

The PR separates scene activation from default content creation across the scene subsystem. The Scene interface in scene.hpp gains clarifying documentation comments on on_enter() and new_scene(). All three scene subclasses (ArtilleryScene, PlatformerScene, SimpleShooterScene) have their on_enter() implementations stripped of the new_scene() call, leaving only a comment stub. App::App and App::run now call m_scene->new_scene() explicitly after SceneFactory::create() and before m_scene->on_enter(). NOTES.md and README.md are updated to reflect the new requirement and completed tasks.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • deanblackborough/Prune#50: Modifies the same App::run() scene loading path in app.cpp, touching the same scene lifecycle flow around load_from_file() and on_enter().

Suggested labels

enhancement

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch release-fixes-multi-select

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

@coderabbitai coderabbitai Bot added the enhancement New feature or request label Jun 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant