Repository navigation
chore: retire the old playground; there is one Playground - #201
Merged
Merged
Conversation
ADR 0004, decision 5. The shell in src/components/playground never ran the learner's code (Run, Test and Submit were timed mocks). - Remove src/components/playground, its hook, scoring, types, memory monitor and theme entry (about 5,700 lines). - Remove the "Interactive Playground" section from the Big-O and Data Structures pages, the sidebar and moduleNavigation. Their Introduction cards now open /playground in a new tab. - Park the problem set and its types under src/features/playground/exercises for the Exercise feature; not wired in. - RxJS: delete the stub code editor (canned output) and rename the rest to the marble diagram tool. - Big-O quiz bank: three questions referenced the removed section. - README and the Playground developer guide describe the one Playground. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Implements decision 5 of ADR 0004: there is exactly one Playground, at
/playground.src/components/playground/was a problem-and-tests shell that never ran the learner's code: Run, Test and Submit were timed mocks. It goes, with its hook, scoring, types, memory monitor and theme entry.Sidebar.tsxandmoduleNavigation.ts. The Introduction cards that pointed at it now open/playgroundin a new tab. An old?section=playgroundlink falls back to Introduction.src/data/problems.tsand the types it needs move, unchanged, tosrc/features/playground/exercises/for the future Exercise feature. Nothing imports them yet.src/components/rxjs/marble/MarbleDiagramTool.tsx), and its copy now says the result is simulated and no RxJS code is run. The sidebar label "Visualization Tool" is unchanged.bigo-q22,q23andq27referenced the removed section; they now reference "Practice Challenges". The scenarios of q22 and q23 no longer mention a playground. Ids and count (31) are unchanged.Checks
npx tsc -b,npm run lint,npm run test:run(33 files, 340 tests) andnpm run buildpass./bigoand/datastructureshave no "Interactive Playground" sidebar entry and their cards link to/playgroundwithtarget="_blank"; the RxJS "Visualization Tool" section renders the Marble Diagram Tool with no console errors. Clicking through the marble tool's operators was not tested.Known, not fixed here
skip,debounceTimeanddistinctUntilChangedare selectable but leave the stream unchanged;mapandfilterrecognise only two expressions each; typing in the editor does not drive the input stream. These predate this PR.debugStateprop that nothing passes now, andAppContext.tsxkeeps an unreadinteractivePlaygroundflag.ComponentPlayground).🤖 Generated with Claude Code