Skip to content

docs: add binary integrity verification section (P0-4)#43

Merged
beonde merged 1 commit into
mainfrom
fix/docs-p0-checksum-docs
May 2, 2026
Merged

docs: add binary integrity verification section (P0-4)#43
beonde merged 1 commit into
mainfrom
fix/docs-p0-checksum-docs

Conversation

@beonde
Copy link
Copy Markdown
Member

@beonde beonde commented May 2, 2026

Summary

Document the checksum verification behavior for the capiscio-core binary download.

Documentation Audit Reference

Phase: P0-4
Severity: Critical

Changes

  • README.md: Added "Binary Integrity Verification" section documenting CAPISCIO_SKIP_CHECKSUM env var
  • README.md: Added troubleshooting entry for checksum verification failures

Verification

  • Code examples verified against src/utils/binary-manager.ts
  • No banned phrases added

Document the checksum verification behavior, CAPISCIO_SKIP_CHECKSUM env var,
and add troubleshooting entry for checksum failures.

Ref: DOCS_REMEDIATION_PLAN P0-4 Part B
Copilot AI review requested due to automatic review settings May 2, 2026 17:57
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds documentation describing how the Node.js wrapper verifies the downloaded capiscio-core binary integrity, and how to troubleshoot checksum-related download failures.

Changes:

  • Adds a “Binary Integrity Verification” README section describing SHA-256 verification and CAPISCIO_SKIP_CHECKSUM.
  • Adds a troubleshooting entry for checksum verification failures.

Comment thread README.md
Comment on lines +83 to +88
If verification fails or the checksums file is unavailable:

```bash
# Temporary bypass (not recommended for production)
export CAPISCIO_SKIP_CHECKSUM=true
```
Comment thread README.md
If you are behind a corporate firewall, ensure you can access `github.com`.

**Checksum verification failures:**
If you see "Checksum verification failed", the binary integrity could not be confirmed.
@beonde beonde merged commit 56743fc into main May 2, 2026
7 checks passed
@beonde beonde deleted the fix/docs-p0-checksum-docs branch May 2, 2026 18:14
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.

2 participants