Em dash cleanup, release-only CI, Sound proposal, Import JS Library#18
Merged
Conversation
…+ JS import button - Replace all em dashes with hyphens across tracked markdown docs. - CMakeLists.txt: set project version to 0.1.0. - .github/workflows/build.yml: only builds + publishes a GitHub Release when a v*.*.* tag is pushed (or manual dispatch) - no longer runs on every push/PR. Day-to-day rounds stay branch -> PR -> merge with no CI gate. Also packages each platform's output into a zip/tar.gz release asset. - docs/proposals/sound-section.md: design proposal for a new "Sound" side panel section (master on/off + 0-1 volume over sketch audio output via a wrapped AudioContext.destination). Not implemented - proposal only. - Add "Import JS Library" button to the Libraries panel: opens a native file picker, reads the chosen .js file's text, and injects it into the sketch iframe (same sandboxed, opaque-origin context as the sketch itself). samples/test-import-lib.js is a minimal file for exercising the button. Co-Authored-By: Claude Sonnet 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
CMakeLists.txt: project version set to0.1.0..github/workflows/build.yml: now only builds + publishes a GitHub Release when av*.*.*tag is pushed (or manual dispatch) - no longer runs on every push/PR. Day-to-day rounds stay branch -> PR -> merge with no CI gate. To cut a release: bump the CMake version,git tag vX.Y.Z && git push --tags.docs/proposals/sound-section.md: design proposal for a "Sound" side-panel section (master on/off toggle + 0-1 volume slider over sketch audio output, via a wrappedAudioContext.destination). Proposal only, not implemented - for review before building..jsfile's text viaFileReader, and injects it into the sketch iframe (same sandboxed, opaque-origin context the sketch itself runs in, so no new attack surface).samples/test-import-lib.jsis a minimal file to exercise it.Test plan
node --check public/script.jspasses.build_and_run.bat); loaded the app's loopback URL in a browser to verify the DOM (sidebar renders the new "Import JS Library" button under Libraries).File/DataTransferdispatched on the file input: hint text updated to "Imported: test-import-lib.js", terminal logged the import, and the sketch auto-reran.SecurityErrorwhen trying to reach into it from the parent), i.e. the import path doesn't weaken isolation.Generated with Claude Code