docs(adr): accepted SHA-256, Whisper, and TMK/VAD decisions - #476
Draft
seonghobae wants to merge 1 commit into
Draft
docs(adr): accepted SHA-256, Whisper, and TMK/VAD decisions#476seonghobae wants to merge 1 commit into
seonghobae wants to merge 1 commit into
Conversation
Add a thin accepted-ADR set for decisions already true on main, with APA 7th references and live locators. Architecture notes stay in place as non-normative supporting detail. Co-authored-by: Seongho Bae <seonghobae@users.noreply.github.com>
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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.
Draft only. Do not merge. Do not mark Ready.
Scope
main(a8e4956). Not stacked on fix(ui): overlay landmarks and click-or-drag copy on drop zones #428, Switch OpenCode to NVIDIA NIM only #431, chore(deps-dev): bump coverage from 7.15.3 to 7.15.4 #408, fix(fuzz): align the Atheris lock with Python 3.14 #331, fix(media): tolerate non-UTF-8 ffmpeg and ffprobe metadata #317, 🎨 Palette: 필수 입력 필드 초기화 시 유효성 검사 접근성 피드백 유지 #455, ⚡ Bolt: [성능 개선] 검색 성능 향상: 불필요한 Set 복사 제거 및 루프 최적화 #453, or any other open feat/UI/deps PR.docs/architecture/).main. Does not invent product behavior.Files
docs/adr/README.md— index; architecture notes become non-normative supporting detail once the ADR is the decision recorddocs/adr/0001-sha256-content-identity.md— full SHA-256 identity; exact duplicates and transcripts keyed by full content hash; unverified placeholders cannot authorize groups or mutationsdocs/adr/0002-pinned-whisper-gpu-transcription.md— pinned MLX/CUDA Whisper revisions; never Ollama; never silent CPU fallback; overlap + timestamp ownershipdocs/adr/0003-tmk-vad-fixed-duration-precedence.md— TMK → chapter/marker → VAD/silence → bounded fixed-duration; five-minute cuts are resource limits; late TMK promote-or-reprocess; quality-gate citations onlyVerified locators (re-fetched live 2026-08-25)
docs/papers/2212.04356-whisper.pdfis a supporting copy only.application/pdf833315 bytes (HEAD of that URL 404’d; GET is live). Figure 1: SHA-256 digest 256 bits, block 512 bits. Repo copydocs/standards/NIST.FIPS.180-4.pdfis a supporting copy only.No required locator was dropped. Not cited: RFC 4122, Gemma model cards, MLX GitHub, fuzzing survey, draft W3C, cancelled scans. MOSS is named only as an implementation pin, not a paper.
Do not merge. Do not mark Ready.