docs: audit real-paper Electron journey - #54
Draft
dkritarth wants to merge 6 commits into
Draft
Conversation
6 tasks
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.
Closes #26
User outcome
Vellum now has evidence-backed current truth for the real Electron journey at
a21d596: the unmodified app fails before ingest, and every reachable result or blocked downstream step is recorded without claiming unit tests as product proof.Scope
Non-goals: this PR fixes no product defect, changes no runtime code, and does not claim reader, Ask, or persistence behavior works.
Acceptance criteria
Fail — blocked.Automated verification
npx vitest run core/acp/stdio-client.test.ts— 18/18 passed.npm test— final run 147/147 across 24 files. Initial run was 146/147 due one ACP disposal timing failure; focused and full reruns passed.npm run typecheck— passed.npm run build— passed and reproduced [S0-AUDIT] Electron dev boot cannot load preload bridge #51 by emittingout/preload/preload.mjswhile main expectsindex.js.git diff --check master...HEAD— passed.Existing jsdom canvas-not-implemented stderr remains test noise and is not treated as live PDF proof.
Live Electron verification
a21d596; confirmed isolateddata/absent and normal checkout data untouched.bridge: …and missing-preload console errors ([S0-AUDIT] Electron dev boot cannot load preload bridge #51).bridge: pongand valid empty reader.Could not load your library.plus native-binding error with preload bypass ([S0-AUDIT] Electron dev boot cannot load preload bridge #51/[S0-AUDIT] Bundled better-sqlite3 cannot load native binding in Electron #52).Exact action transcript:
docs/verification/issue-26/interaction-journal.json.Visual and console evidence
docs/verification/issue-26/01-preload-failure.png— unmodified app,bridge: ….docs/verification/issue-26/02-empty-state.png— empty reader after preload-only bypass,bridge: pong.docs/verification/issue-26/main-console.txt— preload and native-binding errors.docs/verification/issue-26/controller-console.txt— semantic Create click completed withcreateChanged: false.docs/verification/issue-26/last-renderer-console.json— captured post-load renderer window had no page errors after bypass.Real-paper / ACP evidence
Planned paper: arXiv
1706.03762v7, Attention Is All You Need. Title, authors, version, length, and factual grounding targets are sourced from the official arXiv record in the audit report.The paper was not ingested. Signed-in Codex Ask was not reached. Prior issue #25's ACP smoke is not substituted for paper-grounded Electron evidence.
Limitations and follow-ups
better-sqlite3resolves a nonexistent native-addon path.PR remains draft while the two unchecked acceptance criteria remain unmet.
Independent review
status:in-progress, neverstatus:blocked.