Normalize timestamp handling across host and zome#5
Merged
Conversation
kalisam
deleted the
codex/decide-timestamp-representation-for-structs
branch
kalisam
pushed a commit
that referenced
this pull request
Mar 17, 2026
…U3AZewR4jzNMr3Q Phase 1 Automatable Tasks
kalisam
added a commit
that referenced
this pull request
Apr 5, 2026
substrate_bridge.test.ts (Critical + 2 Majors): - Wrap fileURLToPath with path.dirname() so hAppPath resolves correctly (#28) - Add @msgpack/msgpack import + decode record.entry to verify Alice's subject/predicate/object/confidence in Criterion 3 instead of only asserting entry exists (#30) - Add Criterion 3b negative test: get_triple_record returns null for XOR-mutated non-existent hash, then sanity-checks real hash still resolves (#29) ADR-0 (recognition protocol, both copies): - Align participant count: "~7 AI systems" -> "6 AI systems" (#5) - Add Evidence pointers (kernel, ADR index, HARVEST_LOG, etc.) to each PASSED Validation Criterion (#6) - Reclassify harvest-protocol.spec as spec, not ADR-5 itself (#7) - MD022 blank lines after Positive/Negative/Neutral headings (#1/#2) ADR-5 (cognitive-virology-pattern): - Terminal punctuation on Problem Statement bullets (#8) - HARVEST_LOG.md added to Related Documents (#9) .serena/memories: - codebase_structure.md: 5 externs -> 9 with full names, add Infinity Bridge connector path (#10) - project_overview.md: align extern count + current state to 2026-04, PR #25 (#11) - task_completion_checklist.md: expand ADR item with required sections (#14) - style_and_conventions.md, suggested_commands.md: MD022 blank lines around headings + gh CLI now available (#12/#13) .serena/project.yml: - Add markdown + yaml to languages (#15) - Populate initial_prompt with SDD/ADR review checklist (#16) docs/research: - "4MB" -> "4 MB" unit spacing (#18) - Add Query criterion (query_triples without hash) to Phase 0 Gate summary so it matches the 6-criterion spec (#19) - KERI/hREA integration status: Aspirational -> Specified (bridge zomes + entry types exist in identity_integrity/hrea_*) (#21) - cross-ai-orchestration-synthesis: add metadata header marking it as working notes (#22) - Perplexity-Source-Agent-Orchestration: fix 132 mojibake chars (em dash, en dash, superscript 2, not-equal) (#24) - Add truth-status labels to Key New Patterns (AMRO-S, CodeCRDT, AgentNet++, DecentLLMs): External Specified / FLOSSI0ULLK Unverified (#17) docs/specs/phase0-substrate-bridge.spec.md: - Add typescript language specifier to pseudocode fenced blocks (#25) - Resolve provenance design: adopt option (b) - extend TripleResult with author + created_at fields mapped from KnowledgeTriple (#26) - Add provenance-retrieval prerequisite to Definition of Done so the 6-test DoD isn't circular (#27) Deferred: #20 (comparison-table citation sources) and #23 (extract gap matrix as separate artifact) - substantial restructuring work that doesn't block this PR. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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
1 participant
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.
Summary
sys_time()usages in vectors, migration plans, and shard status tracking to millisecondu64valuesCargo.lockto capture the dependency set produced bycargo checkTesting
cargo check(fails: repository currently contains numerous unrelated compilation errors, e.g. missing modules and Holochain types)cargo test --target wasm32-unknown-unknown --manifest-path dna/zomes/fl_core/Cargo.toml(fails: existing zome code uses unsupported Holochain macros/types)Codex Task