Skip to content

feat: add local auto-chart pipeline - #1

Closed
Baltsat wants to merge 1 commit into
codex/import-metadata-truthfrom
codex/sightkick-auto-chart-pipeline-1785232622-6f9a
Closed

feat: add local auto-chart pipeline#1
Baltsat wants to merge 1 commit into
codex/import-metadata-truthfrom
codex/sightkick-auto-chart-pipeline-1785232622-6f9a

Conversation

@Baltsat

@Baltsat Baltsat commented Jul 28, 2026

Copy link
Copy Markdown
Owner

why

SightKick can import prepared charts, but creating one still requires leaving the app and manually operating OCTAVE. This adds the smallest legal, local-first path from a song reference and owned local audio to a reviewed playable chart.

This is stacked on the import/provenance work currently proposed upstream in tonygoldcrest#42. It should be rebased and proposed upstream as one coherent feature after that foundation lands.

what changed

  • optional YouTube video URL for official oEmbed title, artist inference, and thumbnail only
  • explicit local audio picker; no YouTube audio/video download or demux
  • local OCTAVE/STRUM runtime discovery and preflight
  • FIFO processing with progress, cancellation, retry, safe temp paths, and shutdown cleanup
  • review-before-import with cached official artwork and auto-chart provenance
  • imported result immediately appears in the library and opens as rendered drum notation
  • unit, renderer, queue, protocol, and conditional live OCTAVE e2e coverage

proof

  • yarn lint
  • yarn test: 60 files, 715 tests passed
  • yarn build
  • yarn test:e2e:only: 3 passed, 1 conditional live test skipped
  • live Raging proof with official YouTube metadata plus a local 25-second audio file: OCTAVE/STRUM → review → import → rendered score, passed in 40.3 seconds

product boundary

YouTube is discovery metadata only. The user must select audio they own or are allowed to process. CPU/GPU work stays on the Mac; a future shared catalog can store metadata and user-authored chart artifacts without turning free serverless tiers into an unreliable ML worker.

Add a one-click local chart flow with optional official YouTube metadata, OCTAVE/STRUM processing, review-before-import, progress, cancellation, retry, and live end-to-end coverage.

User-Request: встроенный локальный импорт и авто-чарт из SightKick | codex:019f9237-cca9-78e3-a7b7-c821f2bc1581
@gemini-code-assist

Copy link
Copy Markdown

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@coderabbitai

coderabbitai Bot commented Jul 28, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: c9a7e225-7ada-4b8b-938d-ba8edf712b2d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/sightkick-auto-chart-pipeline-1785232622-6f9a

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@Baltsat

Baltsat commented Aug 12, 2026

Copy link
Copy Markdown
Owner Author

Superseded: the auto-chart pipeline shipped in the kb.7-kb.12 line on main (see release notes 1.2.0-kb.9 onward).

@Baltsat Baltsat closed this Aug 12, 2026
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