c5bffb7fe6
Round-2 overhaul of the tool-output surface, built eval-first (byte harness + no-context field-eval agents, always vs the v0.9.0 release binary): Format — tree syntax everywhere: - compact_out emits tree tables: 'key: N (cols: a b c)' headers, indented space-delimited rows, '-' placeholders, space-aware quoting. Every table in every tool converts mechanically. - Grouped prefix-factored output is the DEFAULT for search_graph and trace_path: rows grouped under their shared (qn-prefix, file) printed once, row qn = group prefix + '.' + name (stated in-band). Measured: 192-hit search 2,929B (was 4,811 TOON / 238,466 v0.9.0), 82-row trace 2,256B (was 4,234 / 8,094). Field eval: 5/5 answers correct, QN reconstruction verified, no comprehension loss. - detail:'ids' tier: bare-qn enumeration for wide sweeps (-44%). Pagination — exactly-once, stateless: - store_meta generation (per-DB random uid + mutation counter bumped in cbm_store_upsert_project — every index run's choke point): cursors can never silently validate against a rebuilt DB. - trace cursors: opaque token (leg, generation, params-hash, (hop,id) watermark); pages drain callees then callers with exact callees_total/callers_total on every page; teaching errors for params-mismatch and staleness. Guard: 12 callees paged 5+5+2, every row on exactly one page. - deterministic ordering: BFS ORDER BY hop,id; canonical (hop,id) sort + min-hop dedup across same-name seeds (hop fed risk_labels wrong before); search (name,id)/(rank,id) tie-breaks make offset pages stable. Lean defaults + teaching errors: - get_graph_schema no longer advertises blocked internal fields (fp/sp/bt). - index_repository coverage lists: counts + 5 examples + one-line pointer notes (full record was always in the logfile); 6.0KB -> 1.5KB here, 53KB observed on a large repo. index_status gates the git context block behind verbose:true. - trace limit param (<=5000) + truncated:true — never a silent truncation; invalid direction now errors with the valid values; requesting core columns via fields hints instead of emitting empty cells; cypher resolves f.file/f.qn/f.path aliases. Trace-correctness fixes (reproduce-first, RED verified): - bfs_collect_edges 4KB id-string overflow: past ~1000 visited nodes the edge SQL failed to prepare and EVERY trace edge silently vanished; now a per-connection temp-table join (star guard: 1200/1200 edges). - bfs_union_same_name: min-hop across seeds + edge dedup. Guards: gating tool-output regression suite (context-explosion detector: blocked-field leaks, repeated-key envelopes, oversized notes + calibrated byte ceilings + semantic floors). Full suite 6,367 passed / 0 failed. Signed-off-by: Martin Vogel <martin.vogel.tech@gmail.com>
248 lines
9.3 KiB
C
248 lines
9.3 KiB
C
/*
|
|
* test_store_pragmas.c — Tests for SQLite pragma resolution.
|
|
*
|
|
* Validates that the CBM_SQLITE_MMAP_SIZE env var controls the mmap_size
|
|
* pragma applied to on-disk stores. Default behavior (env unset) must
|
|
* remain 64 MB. Setting the env to 0 disables memory-mapped I/O so
|
|
* concurrent processes that truncate the DB file under a sibling's live
|
|
* mapping return SQLITE_IOERR instead of crashing the process with SIGBUS.
|
|
*/
|
|
#include "../src/foundation/compat.h"
|
|
#include "test_framework.h"
|
|
#include "test_helpers.h"
|
|
#include <store/store.h>
|
|
#include <stdio.h>
|
|
#include <stdlib.h>
|
|
#include <string.h>
|
|
#include <unistd.h>
|
|
|
|
static void clear_mmap_env(void) {
|
|
cbm_unsetenv("CBM_SQLITE_MMAP_SIZE");
|
|
}
|
|
|
|
TEST(mmap_size_default_when_unset) {
|
|
clear_mmap_env();
|
|
ASSERT_EQ(cbm_store_resolve_mmap_size(), 67108864LL);
|
|
PASS();
|
|
}
|
|
|
|
TEST(mmap_size_zero_disables_mmap) {
|
|
cbm_setenv("CBM_SQLITE_MMAP_SIZE", "0", 1);
|
|
ASSERT_EQ(cbm_store_resolve_mmap_size(), 0LL);
|
|
clear_mmap_env();
|
|
PASS();
|
|
}
|
|
|
|
TEST(mmap_size_explicit_value) {
|
|
cbm_setenv("CBM_SQLITE_MMAP_SIZE", "1048576", 1);
|
|
ASSERT_EQ(cbm_store_resolve_mmap_size(), 1048576LL);
|
|
clear_mmap_env();
|
|
PASS();
|
|
}
|
|
|
|
TEST(mmap_size_negative_clamped_to_zero) {
|
|
cbm_setenv("CBM_SQLITE_MMAP_SIZE", "-1", 1);
|
|
ASSERT_EQ(cbm_store_resolve_mmap_size(), 0LL);
|
|
clear_mmap_env();
|
|
PASS();
|
|
}
|
|
|
|
TEST(mmap_size_garbage_falls_back_to_default) {
|
|
cbm_setenv("CBM_SQLITE_MMAP_SIZE", "not-a-number", 1);
|
|
ASSERT_EQ(cbm_store_resolve_mmap_size(), 67108864LL);
|
|
clear_mmap_env();
|
|
PASS();
|
|
}
|
|
|
|
TEST(mmap_size_partial_garbage_falls_back_to_default) {
|
|
cbm_setenv("CBM_SQLITE_MMAP_SIZE", "123abc", 1);
|
|
ASSERT_EQ(cbm_store_resolve_mmap_size(), 67108864LL);
|
|
clear_mmap_env();
|
|
PASS();
|
|
}
|
|
|
|
/* Integration smoke: opening a file-backed store with mmap_size=0 must
|
|
* succeed. Proves the resolver is wired through configure_pragmas(). */
|
|
TEST(store_open_with_mmap_disabled) {
|
|
cbm_setenv("CBM_SQLITE_MMAP_SIZE", "0", 1);
|
|
char tmp_path[256];
|
|
snprintf(tmp_path, sizeof(tmp_path), "%s/cbm_test_pragmas_%d.db", cbm_tmpdir(), (int)getpid());
|
|
unlink(tmp_path);
|
|
|
|
cbm_store_t *s = cbm_store_open_path(tmp_path);
|
|
ASSERT(s != NULL);
|
|
cbm_store_close(s);
|
|
|
|
unlink(tmp_path);
|
|
/* WAL/SHM siblings created by the open */
|
|
char tmp_wal[300];
|
|
char tmp_shm[300];
|
|
snprintf(tmp_wal, sizeof(tmp_wal), "%s-wal", tmp_path);
|
|
snprintf(tmp_shm, sizeof(tmp_shm), "%s-shm", tmp_path);
|
|
unlink(tmp_wal);
|
|
unlink(tmp_shm);
|
|
|
|
clear_mmap_env();
|
|
PASS();
|
|
}
|
|
|
|
/* #1083: on-disk write connections must bound the WAL via journal_size_limit
|
|
* so a checkpoint-starved log is physically reclaimed once a checkpoint can
|
|
* reset it. On main this is UNSET (-1 = unlimited), so the -wal file only ever
|
|
* grows (all our checkpoints are PASSIVE and never ftruncate). Read the pragma
|
|
* back on the SAME connection — it's per-connection and not persisted. */
|
|
TEST(journal_size_limit_bounds_wal_issue1083) {
|
|
char tmp_path[256];
|
|
snprintf(tmp_path, sizeof(tmp_path), "%s/cbm_test_jsl_%d.db", cbm_tmpdir(), (int)getpid());
|
|
unlink(tmp_path);
|
|
|
|
cbm_store_t *s = cbm_store_open_path(tmp_path);
|
|
ASSERT(s != NULL);
|
|
/* 256 MiB — far above the healthy WAL (~4 MiB), so no truncate/regrow churn
|
|
* in normal operation; it only fires after abnormal (starved) growth. */
|
|
ASSERT(cbm_store_journal_size_limit(s) == (int64_t)268435456);
|
|
cbm_store_close(s);
|
|
|
|
unlink(tmp_path);
|
|
char tmp_wal[300];
|
|
char tmp_shm[300];
|
|
snprintf(tmp_wal, sizeof(tmp_wal), "%s-wal", tmp_path);
|
|
snprintf(tmp_shm, sizeof(tmp_shm), "%s-shm", tmp_path);
|
|
unlink(tmp_wal);
|
|
unlink(tmp_shm);
|
|
PASS();
|
|
}
|
|
|
|
/* Pagination-cursor generation: minted per DB file, bumped per index run.
|
|
* Same store + reads only -> stable; upsert_project (every index run's choke
|
|
* point) -> changes; two distinct DB files can never share a generation even
|
|
* at the same counter value (random db_uid). */
|
|
TEST(store_generation_tracks_mutations) {
|
|
char g1[128];
|
|
char g2[128];
|
|
char g3[128];
|
|
cbm_store_t *a = cbm_store_open_memory();
|
|
ASSERT(a != NULL);
|
|
ASSERT_EQ(cbm_store_upsert_project(a, "p", "/tmp/p"), CBM_STORE_OK);
|
|
ASSERT_EQ(cbm_store_generation(a, g1, sizeof(g1)), CBM_STORE_OK);
|
|
ASSERT(strncmp(g1, "u", 1) == 0); /* seeded, not legacy */
|
|
ASSERT_EQ(cbm_store_generation(a, g2, sizeof(g2)), CBM_STORE_OK);
|
|
ASSERT(strcmp(g1, g2) == 0); /* reads are stable */
|
|
ASSERT_EQ(cbm_store_upsert_project(a, "p", "/tmp/p"), CBM_STORE_OK);
|
|
ASSERT_EQ(cbm_store_generation(a, g3, sizeof(g3)), CBM_STORE_OK);
|
|
ASSERT(strcmp(g1, g3) != 0); /* index run bumps */
|
|
|
|
cbm_store_t *b = cbm_store_open_memory();
|
|
ASSERT(b != NULL);
|
|
ASSERT_EQ(cbm_store_upsert_project(b, "p", "/tmp/p"), CBM_STORE_OK);
|
|
char gb[128];
|
|
ASSERT_EQ(cbm_store_generation(b, gb, sizeof(gb)), CBM_STORE_OK);
|
|
ASSERT(strcmp(g1, gb) != 0); /* distinct DBs never alias (random uid) */
|
|
cbm_store_close(a);
|
|
cbm_store_close(b);
|
|
PASS();
|
|
}
|
|
|
|
/* #896: a row-scan that dies mid-stream (SQLITE_CORRUPT) must surface a
|
|
* loud store error, not masquerade as a clean end of results. Counts are
|
|
* answered from covering indexes (still correct) while row fetches die at
|
|
* the first corrupt table page — the old loops discarded the terminal
|
|
* sqlite3_step code, so every query surface returned plausible
|
|
* truncated/empty answers with no error. */
|
|
TEST(corrupt_page_scan_returns_error_not_truncation) {
|
|
enum { CORRUPT_NODES = 2000, ZERO_PAGES = 40 };
|
|
char *td = th_mktempdir("cbm_corrupt");
|
|
char db_path[512];
|
|
snprintf(db_path, sizeof(db_path), "%s/c.db", td);
|
|
|
|
cbm_store_t *s = cbm_store_open_path(db_path);
|
|
ASSERT_NOT_NULL(s);
|
|
cbm_store_upsert_project(s, "corr", "/tmp/corr");
|
|
for (int i = 0; i < CORRUPT_NODES; i++) {
|
|
char name[64];
|
|
char qn[256];
|
|
snprintf(name, sizeof(name), "corrupt_probe_fn_%04d", i);
|
|
snprintf(qn, sizeof(qn),
|
|
"corr.some.rather.long.module.path.to.fill.table.pages.%s_padding_padding", name);
|
|
cbm_node_t n = {.project = "corr",
|
|
.label = "Function",
|
|
.name = name,
|
|
.qualified_name = qn,
|
|
.file_path = "src/corrupt_probe.py",
|
|
.start_line = i + 1,
|
|
.end_line = i + 2};
|
|
ASSERT_TRUE(cbm_store_upsert_node(s, &n) > 0);
|
|
}
|
|
/* Precondition: a full scan works on the healthy file. */
|
|
cbm_search_params_t params = {.project = "corr", .label = "Function", .limit = 50};
|
|
cbm_search_output_t out = {0};
|
|
ASSERT_EQ(cbm_store_search(s, ¶ms, &out), CBM_STORE_OK);
|
|
ASSERT_EQ(out.total, CORRUPT_NODES);
|
|
cbm_store_search_free(&out);
|
|
cbm_store_close(s);
|
|
|
|
/* Zero a band of mid-file pages (the report's dd repro): page 25%..
|
|
* covers nodes-table leaves on a file this shape. */
|
|
FILE *f = fopen(db_path, "rb+");
|
|
ASSERT_NOT_NULL(f);
|
|
(void)fseek(f, 0, SEEK_END);
|
|
long fsize = ftell(f);
|
|
enum { PAGE = 4096 };
|
|
long page_count = fsize / PAGE;
|
|
ASSERT_TRUE(page_count > ZERO_PAGES + 8);
|
|
char zero[PAGE];
|
|
memset(zero, 0, sizeof(zero));
|
|
(void)fseek(f, (page_count / 4) * (long)PAGE, SEEK_SET);
|
|
for (int i = 0; i < ZERO_PAGES; i++) {
|
|
ASSERT_EQ(fwrite(zero, 1, PAGE, f), (size_t)PAGE);
|
|
}
|
|
(void)fclose(f);
|
|
|
|
/* The scans must now fail LOUDLY (CBM_STORE_ERR), not truncate. */
|
|
cbm_store_t *s2 = cbm_store_open_path(db_path);
|
|
ASSERT_NOT_NULL(s2);
|
|
/* The scan must CROSS the corrupt band: request every row. */
|
|
cbm_search_params_t all_params = {
|
|
.project = "corr", .label = "Function", .limit = CORRUPT_NODES};
|
|
cbm_search_output_t out2 = {0};
|
|
int rc_search = cbm_store_search(s2, &all_params, &out2);
|
|
if (rc_search == CBM_STORE_OK && out2.count == CORRUPT_NODES) {
|
|
/* Vacuous-guard: a complete, healthy scan means corruption missed
|
|
* the table pages — rebuild the fixture, don't relax the assert. */
|
|
FAIL("fixture failed to hit table pages (full scan healthy)");
|
|
}
|
|
/* THE BUG (#896): OK + silently truncated rows. Fixed = loud ERR. */
|
|
ASSERT_EQ(rc_search, CBM_STORE_ERR);
|
|
cbm_store_search_free(&out2);
|
|
|
|
/* Point lookups may legitimately succeed when their row's page
|
|
* escaped the corrupt band — the class contract is about SCANS. A
|
|
* second scan surface (qn-suffix, different SQL path) must also err. */
|
|
cbm_node_t *hits = NULL;
|
|
int hit_count = 0;
|
|
int rc_suffix =
|
|
cbm_store_find_nodes_by_qn_suffix(s2, "corr", "padding_padding", &hits, &hit_count);
|
|
if (rc_suffix == CBM_STORE_OK && hit_count == CORRUPT_NODES) {
|
|
FAIL("suffix scan healthy — fixture failed to hit table pages");
|
|
}
|
|
ASSERT_EQ(rc_suffix, CBM_STORE_ERR);
|
|
cbm_store_free_nodes(hits, hit_count);
|
|
cbm_store_close(s2);
|
|
|
|
unlink(db_path);
|
|
PASS();
|
|
}
|
|
|
|
SUITE(store_pragmas) {
|
|
RUN_TEST(journal_size_limit_bounds_wal_issue1083);
|
|
RUN_TEST(store_generation_tracks_mutations);
|
|
RUN_TEST(corrupt_page_scan_returns_error_not_truncation);
|
|
RUN_TEST(mmap_size_default_when_unset);
|
|
RUN_TEST(mmap_size_zero_disables_mmap);
|
|
RUN_TEST(mmap_size_explicit_value);
|
|
RUN_TEST(mmap_size_negative_clamped_to_zero);
|
|
RUN_TEST(mmap_size_garbage_falls_back_to_default);
|
|
RUN_TEST(mmap_size_partial_garbage_falls_back_to_default);
|
|
RUN_TEST(store_open_with_mmap_disabled);
|
|
}
|