feat(ffi): Zig FFI consumer demo + CI lane (#6)#13
Merged
Conversation
De-templates ffi/zig from the {{PROJECT}} skeleton and proves the C-ABI
contract with a linking consumer.
- build.zig: ported to the Zig 0.15.x module API (b.addLibrary /
b.createModule), dropped the dead header/bench/docs/integration steps
that referenced nonexistent files, link_libc (the FFI uses
std.heap.c_allocator), and added `test` / `consumer` / `check` steps.
- src/main.zig: added the OctadDimension + ProvenanceEntry extern
structs and verisimdb_data_{octad,provenance}_{encode,decode} with a
lossless wire format. Fixed two latent template bugs that blocked any
build: `Handle` was `opaque` *with fields* (now a struct; still
opaque to C behind ?*Handle), and `callconv(.C)` → `callconv(.c)`
(0.15 rename). Added 5 round-trip/negative unit tests.
- test/octad_consumer.zig: NEW standalone executable that
linkLibrary()s the FFI and round-trips both structs across the
boundary (incl. a short-buffer negative path); exits non-zero on any
mismatch. Replaces test/integration_test.zig, which was an
uncompilable {{project}} template stub.
- .github/workflows/zig-ffi.yml: CI lane; Zig pinned by tarball
sha256 (no third-party action — Scorecard pinned-deps clean) running
`zig build check`.
Local: `zig build check` → 7/7 unit tests pass; consumer prints
"PASS: verisimdb-data FFI v0.1.0 — OctadDimension + ProvenanceEntry
round-trip OK".
Closes #6.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
🔍 Hypatia Security ScanFindings: 12 issues detected
View findings[
{
"reason": "No test directory or test files found",
"type": "no_tests",
"file": "/home/runner/work/verisimdb-data/verisimdb-data",
"action": "flag",
"rule_module": "honest_completion",
"severity": "high",
"deduction": 20
},
{
"reason": "Issue in quality.yml",
"type": "missing_workflow",
"file": "quality.yml",
"action": "create",
"rule_module": "workflow_audit",
"severity": "high"
},
{
"reason": "Issue in security-policy.yml",
"type": "missing_workflow",
"file": "security-policy.yml",
"action": "create",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action hyperpolymath/standards/.github/workflows/governance-reusable.yml@main needs attention",
"type": "unpinned_action",
"file": "governance.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "high"
},
{
"reason": "codeql.yml lists `language: javascript-typescript` but the repo has no source files in any CodeQL-scannable language. The analyze job will exit 'no source files' on every run. Switch the matrix to `actions` (which scans workflow files — every repo has those).",
"type": "codeql_language_matrix_mismatch",
"file": "codeql.yml",
"action": "switch_codeql_matrix_to_actions",
"rule_module": "workflow_audit",
"severity": "high"
},
{
"reason": "Download-and-execute pattern (curl|wget pipe to shell) -- verify integrity before execution (3 occurrences, CWE-494)",
"type": "shell_download_then_run",
"file": "/home/runner/work/verisimdb-data/verisimdb-data/setup.sh",
"action": "flag",
"rule_module": "code_safety",
"severity": "high"
},
{
"reason": "Nominal-only SAST in verisimdb-data: codeql.yml language matrix contains no language present in the repo and lacks `actions`, so CodeQL records zero results on every commit. Remediation: set the CodeQL matrix to `language: actions`.",
"type": "StaticAnalysis",
"file": "/home/runner/work/verisimdb-data/verisimdb-data",
"action": "auto_fix",
"rule_module": "scorecard",
"severity": "medium",
"remediation": "Add CodeQL or equivalent SAST workflow.",
"scorecard_check": "SAST"
},
{
"reason": "Repository has 4 non-main remote branch(es). Policy: single main branch only.",
"type": "GS007",
"file": ".",
"action": "delete_remote_branches",
"rule_module": "git_state",
"severity": "medium"
},
{
"reason": "References STATE.scm -- should be .machine_readable/6a2/STATE.a2ml",
"type": "SD007",
"file": "0-AI-MANIFEST.a2ml",
"action": "update_reference",
"rule_module": "structural_drift",
"severity": "medium"
},
{
"reason": "References META.scm -- should be .machine_readable/6a2/META.a2ml",
"type": "SD007",
"file": "0-AI-MANIFEST.a2ml",
"action": "update_reference",
"rule_module": "structural_drift",
"severity": "medium"
}
]Powered by Hypatia Neurosymbolic CI/CD Intelligence |
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.
Resolves #6 (V-L3-N1) — add a Zig FFI consumer demo + CI lane.
The
ffi/zigtree was an un-buildable{{PROJECT}}skeleton (pre-0.14addSharedLibraryAPI, references to nonexistentinclude/{{project}}.h/bench/bench.zig, anopaqueHandlewith fields,callconv(.C), and a{{project}}integration-test stub). This de-templates it and proves the contract end-to-end.Changes
build.zig— ported to the Zig 0.15.x module API (b.addLibrary/b.createModule),link_libc(the FFI usesstd.heap.c_allocator), dropped the dead header/bench/docs/integration steps, addedtest/consumer/checksteps.src/main.zig— addedOctadDimension+ProvenanceEntryextern structs andverisimdb_data_{octad,provenance}_{encode,decode}with a lossless wire format; fixed the two latent template bugs that blocked any build (Handleopaque→struct,callconv(.C)→.c); +5 round-trip/negative unit tests.test/octad_consumer.zig— NEW standalone executable thatlinkLibrary()s the FFI and round-trips both structs across the boundary (incl. a short-buffer negative path), non-zero exit on mismatch. Replaces the uncompilabletest/integration_test.zigstub..github/workflows/zig-ffi.yml— CI lane; Zig pinned by tarball sha256 (no third-party action → Scorecard pinned-deps clean) runningzig build check..gitignore— ignoreffi/zig/.zig-cache/+zig-out/.Acceptance
ffi/zig/test/(octad_consumer.zig, links the library)zig-ffi.yml→zig build check)Local verification
🤖 Generated with Claude Code