feat(roles): wire measured register overlap into section warnings - #873
Draft
cursor[bot] wants to merge 56 commits into
Draft
feat(roles): wire measured register overlap into section warnings#873cursor[bot] wants to merge 56 commits into
cursor[bot] wants to merge 56 commits into
Conversation
- 레지스터 중복 감지 이중 루프를 필터-루프로 개선하여 O(N^2) 성능 오버헤드 완화 - 대형 배열 오디오 처리에 의한 Unbounded Memory Consumption 완화 위해 사이즈 제한 적용 - trivy-fs 스캔에서 보고된 pdfjs-dist 취약점 패치 및 예외 처리
- 레지스터 중복 감지 이중 루프를 필터-루프로 개선하여 O(N^2) 성능 오버헤드 완화 - 대형 배열 오디오 처리에 의한 Unbounded Memory Consumption 완화 위해 최대 1억 샘플 사이즈 제한 적용 - 수많은 stem 인풋으로 인한 Denial of Service 방지를 위해 최대 stem 갯수 100개 제한 적용 - trivy-fs 스캔에서 보고된 pdfjs-dist 취약점 패치 및 예외 처리
- 레지스터 중복 감지 이중 루프를 필터-루프로 개선하여 O(N^2) 성능 오버헤드 완화 - 대형 배열 오디오 처리에 의한 Unbounded Memory Consumption 완화 위해 최대 1억 샘플 사이즈 제한 적용 - 수많은 stem 인풋으로 인한 Denial of Service 방지를 위해 최대 stem 갯수 100개 제한 적용 - trivy-fs 스캔에서 보고된 pdfjs-dist 취약점 패치 및 예외 처리
- 레지스터 중복 감지 이중 루프를 필터-루프로 개선하여 O(N^2) 성능 오버헤드 완화 - 대형 배열 오디오 처리에 의한 Unbounded Memory Consumption 완화 위해 최대 1억 샘플 사이즈 제한 적용 - 수많은 stem 인풋으로 인한 Denial of Service 방지를 위해 최대 stem 갯수 100개 제한 적용 - trivy-fs 스캔에서 보고된 pdfjs-dist 취약점 패치 및 예외 처리
- 레지스터 중복 감지 이중 루프를 필터-루프로 개선하여 O(N^2) 성능 오버헤드 완화 - 대형 배열 오디오 처리에 의한 Unbounded Memory Consumption 완화 위해 최대 10,000,000 샘플 사이즈 제한 적용 - 수많은 stem 인풋으로 인한 Denial of Service 방지를 위해 최대 stem 갯수 10개 제한 적용 - trivy-fs 스캔에서 보고된 pdfjs-dist 취약점 패치 및 예외 처리
Revert the latest branch-wide dependency, Trivy, Jules-note, and altered resource-limit drift. Preserve the previously reviewed two-file register-overlap optimization and its focused resource-guard regressions; coordinated dependency security remains owned by #783.
Drop the Bolt-head 10M-sample and 10-stem silent guards, the Trivy ignore, and the npm/PDF.js lock churn so this change stays an overlap optimization. Keep fail-closed invalid thresholds and the policy-owned admission regressions. Co-authored-by: Seongho Bae <seonghobae@users.noreply.github.com>
Stop fabricating keyboard and vocal clash copy. Slice admitted stems to each section, format honest accompaniment labels, and refresh rehearsal priority from measured occupancy. Co-authored-by: Seongho Bae <seonghobae@users.noreply.github.com>
Keep the 80 Hz verse / 1 kHz chorus fixture from dropping HIGH/MEDIUM priority when warnings attach or clear, and retire the leftover Melodic overlap phrase in the priority unit test. Co-authored-by: Seongho Bae <seonghobae@users.noreply.github.com>
This was referenced Aug 20, 2026
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.
Canonical buyer-visible slice
This Draft replaces fabricated role-overlap copy with section-windowed measured overlap, keeps mixed htdemucs
otheroff named keyboard/guitar identity, and preserves the isolated active-stem optimization.Protected base:
develop@acdbea6344fe1231c39535b575f4de35e4c607c9.Exact current head:
9423ae62492fd9bc3530f9a603bbd5b864322a21.Product / evidence contract
otherremains accompaniment and cannot authorize Keyboard Left Hand, Keyboard Right Hand, or Acoustic Guitar identity/presence.Test-first current-head repairs
Boolean temporal/sample-rate authority
Exact-code review found Python
boolvalues could pass ordinary numeric checks becauseboolis anintsubtype:sr=Truecould enter FFT frequency construction, and Boolean section start/end/sample-rate values could become 0/1 timing authority.2b2ca42f1bbfa604943177aad124e9b26b1d7e11adds direct Boolean sample-rate and section-window regressions.2db7a4797e57f4271bae260efc090fa99e51692frejects Boolean sample rate inband_energy_profileand Boolean start/end/rate evidence inslice_stems_to_window, returning zero/empty evidence without I/O authority.e7ef5eadbaf1db8809318a0938467e03d6e76621records the repair in CHANGELOG.9423ae62492fd9bc3530f9a603bbd5b864322a21alignsdocs/doctoring/register-overlap.mdwith the exact trust/test boundary and preserves the APA 7 psychoacoustic/MIR references.Earlier branch repairs remain current: malformed multichannel stems fail closed before FFT; non-finite direct sample rates fail closed; finite raw timing whose scaled sample-index product overflows fails closed; matching result ordering and role-honesty are preserved.
Exact-current-head verification
The first workflow generation for
9423ae62492fd9bc3530f9a603bbd5b864322a21was cancelled before repository jobs executed. That cancellation was inspected rather than treated as source failure. Exact-head reruns have now completed on the unchanged head.Terminal-success repository workflows:
cirun31972445814;secret-scan-gate31972445831;SAST Semgrep31972445850;release31972445830;bandit31972445853;build-baseline31972445824; andsbom31972445862.Two exact-head failures were inspected at the job/step/log boundary and are inherited protected-base dependency evidence rather than changes introduced by this branch:
security-auditrun31972445819, attempt 2, job95230076341completed checkout/tool setup and failed specifically atAudit npm dependencies; subsequent Python/Rust audit stages were skipped. This branch has no JavaScript dependency/root-lock delta.Security Scanrun31972445818, attempt 2 failed only intrivy-fsjob95230123983.dependency-reviewandosv-scansucceeded, including the base/head PR-introduced comparison. Trivy reported exactly one HIGH whole-tree finding:CVE-2026-16633forpdfjs-distatpackage-lock.json:6370, and the workflow itself instructs remediation on the shared base branch.Therefore neither failure is suppressed, copied into this branch, or counted as success. Canonical #783 remains the dependency-root owner. Rerunning these unchanged failing gates again would only reproduce the same protected-base finding.
Dependency / merge gate
This branch changes no JavaScript dependency/root lock or vulnerability suppression. Protected-base dependency findings remain canonical #783-owned and must never be suppressed or copied here. Python statement+branch merge evidence remains #861-owned. The current central required-workflow replacement is read-only
.github#1020; historical #1008 is closed unmerged.Keep Draft and unmerged until the unchanged then-current head has realistic focused/full tests GREEN, exact 100% owned production statement/branch coverage and public docstrings, applicable numerical parity, all required repository/central SAST/security/SBOM/supply-chain/package/release gates terminal-success, zero valid unresolved findings, dependency prerequisites integrated/regenerated, a qualifying independent non-author last-push approval, and ordinary branch-protection acceptance without bypass. Never count queued, cancelled, skipped-required, failed, predecessor-head, protected-base, model-only, self/author, or bypass evidence as success.