bigmerge
678dac5efc
First concern moved out of el_runtime.c under the ratchet, and the move is
deliberately small: it exists to prove the mechanism end to end before anything
large depends on it.
engram_text.{c,h} — query tokenization, candidate-token hygiene, word-boundary
matching, and the text-damage signature. Four functions, moved verbatim; only
`static` was dropped and each doc comment travelled with the code. They touch no
EL value type and no engram store type: plain C over <ctype.h>/<string.h> over
char buffers. They were never el_runtime.c's business.
el_runtime.c 20,527 -> 20,427 lines (BUDGET max_lines ratcheted down)
engram fns 279 -> 275 (BUDGET max_engram_fns ratcheted down)
The Stage 1 extension point worked as designed: adding the file to
lang/runtime/SOURCES was one line, and every build path picked it up. The
Stage 2 drift guard then caught that I had NOT added it to install.sh's
standalone list — the exact class of drift it was written for, on its first
real change, before the commit rather than after a broken SDK shipped.
WHY ONLY 100 LINES, AND WHAT ACTUALLY BLOCKS THE REST
Measured, not estimated: of 273 engram-domain functions in el_runtime.c
(~9,700 lines), only 75 (~1,058 lines) can move today, and they are scattered
rather than clustered. The blocker is a single fact:
EngramNode, EngramEdge, EngramStore, EngramLayer, EngramWal and EngramIdSlot
are typedef'd INSIDE el_runtime.c. No sibling can see them. engram_store.h
defines a SEPARATE serializable "node view" struct and maps between the two.
So every engram function that takes an EngramNode* — which is most of them, 109
of 273 by direct type reference — cannot compile in engram_store.c until those
types move to a shared header. That extraction is the real Stage 3 enabler and
it deserves its own change: it touches the most load-bearing struct in the
system, and doing it in the same commit as a code move would make a regression
impossible to bisect.
REPAIRED: 10 engram harnesses that had silently stopped linking
Not new breakage from this move — verified against unmodified dev, where
el_runtime.c + engram_store.c alone already failed with undefined symbols.
They had been dead for as long as el_runtime.c has been calling into the
siblings, and nothing noticed because nothing ran them.
run_m3_parity, run_m7_traversal, run_m35_hebb_persist,
run_interoception_p0..p5 — now build from $(scripts/el-runtime-sources.sh)
run_wal_tests — its two TUs #include "el_runtime.c" directly, so
it links the SIBLINGS ONLY; adding el_runtime.c
to that link line would define every symbol twice
(That #include'd .c is worth recording: the runtime does have one, in
engram/test/test_wal.c and the generated test_failloud.c.)
Verified locally — every one of these was run, not assumed:
* m3_parity ............ PASS, incl. ASan+UBSan clean across seed/on/reboot
* m7_traversal ......... PASS
* m35_hebb_persist ..... PASS (the gate over the original prod hebb bug)
* interoception p0..p5 . PASS (all six)
* wal_tests ............ 66 passed, 0 failed, + fail-loud exit check
* self-host fixpoint ... byte-identical, AND the emitted C is byte-identical
to the pre-move compiler output — the move changes
nothing the compiler produces
* engram/src/server.el . compiles and links
* native suites ........ 8 of 13, unchanged from before the move; the same 5
pre-existing failures, no regression
* both runtime guards .. green at the new, lower budget
Also fixes a block comment left unterminated by the extraction (the deleted
range carried its closing */), restoring the compile to its single pre-existing
-Wcomment warning.
58 lines
2.5 KiB
Plaintext
58 lines
2.5 KiB
Plaintext
# SOURCES — the canonical El runtime link set.
|
|
#
|
|
# THIS FILE IS THE SINGLE SOURCE OF TRUTH for "what do I compile and link to
|
|
# get the El runtime". Every build path — CI, install.sh, the SDK release, the
|
|
# docs, elb, the engram test harnesses — reads it via scripts/el-runtime-sources.sh
|
|
# instead of hardcoding its own list.
|
|
#
|
|
# WHY THIS FILE EXISTS
|
|
# --------------------
|
|
# The runtime has been multi-translation-unit since the engram siblings landed:
|
|
# el_runtime.c #includes engram_{store,vindex,geometry,reason,verify,cognition}.h
|
|
# and makes hard cross-TU calls into all six. Linking el_runtime.c ALONE has been
|
|
# broken since then — `ld` fails with undefined symbols (engram_ground_json,
|
|
# engram_activate_inner, eg_find_relation, cog_assert_two_axis, ...).
|
|
#
|
|
# It stayed broken because the link set was written out longhand in ~8 different
|
|
# places, each of which drifted independently. A list copied 8 times is a list
|
|
# that is wrong in 8 places. It is now written once, here.
|
|
#
|
|
# HOW TO USE IT
|
|
# -------------
|
|
# scripts/el-runtime-sources.sh # bare names, one per line
|
|
# scripts/el-runtime-sources.sh lang/runtime # prefixed with a directory
|
|
# cc ... $(scripts/el-runtime-sources.sh lang/runtime) -lcurl -lssl -lcrypto -lpthread -lm
|
|
#
|
|
# ADDING A FILE
|
|
# -------------
|
|
# Add the .c here and it is picked up by every build path at once. That is the
|
|
# point: a new concern gets its own translation unit and costs one line, instead
|
|
# of being appended to el_runtime.c because appending was the cheaper edit.
|
|
#
|
|
# Order is link order. Blank lines and `#` comments are ignored.
|
|
|
|
# --- EL core language runtime -------------------------------------------------
|
|
el_runtime.c
|
|
el_seed.c
|
|
|
|
# --- Engram: store, index, geometry, reasoning, verification, cognition -------
|
|
# These are the six concern-owned translation units el_runtime.c calls into.
|
|
engram_store.c
|
|
engram_vindex.c
|
|
engram_geometry.c
|
|
engram_reason.c
|
|
engram_verify.c
|
|
engram_cognition.c
|
|
|
|
# --- Text: tokenization, token hygiene, damage signature ---------------------
|
|
# Extracted from el_runtime.c 2026-08-16. Plain C over <ctype.h>/<string.h> —
|
|
# touches no EL value type and no engram store type. New text helpers go HERE.
|
|
engram_text.c
|
|
|
|
# --- Vector math: batch cosine + its CPU strategy ----------------------------
|
|
# The ggml strategy (eg_cosine_batch_strategy_ggml.c) is an OPTIONAL swap-in and
|
|
# is deliberately NOT in the default set — it needs ggml headers. Link it in
|
|
# place of the cpu strategy when you have them.
|
|
eg_cosine_batch.c
|
|
eg_cosine_batch_strategy_cpu.c
|