From b735d272229b9e74c7d95d952b17d791dc780bea Mon Sep 17 00:00:00 2001 From: Aaron Elijah Mars <61592645+aaronjmars@users.noreply.github.com> Date: Wed, 29 Jul 2026 13:24:51 -0400 Subject: [PATCH 1/8] optimize: make structure tests assert instead of print (gate fix) Every check was computed, logged, then discarded; each test function ended in an unconditional 'return true' and runAllTests never set an exit code. CI's Structure tests step could not fail, so deleting the body of background.js kept CI green. Checks now fold into the return value and a failure exits 1. Also collapses the three near-identical per-script test functions into one table-driven check. --- opendia-extension/test-extension.js | 243 ++++++++++++---------------- 1 file changed, 106 insertions(+), 137 deletions(-) diff --git a/opendia-extension/test-extension.js b/opendia-extension/test-extension.js index f186dcc..2d2f736 100644 --- a/opendia-extension/test-extension.js +++ b/opendia-extension/test-extension.js @@ -1,182 +1,151 @@ -// Simple test script to verify browser extension compatibility +// Structural checks over the built extensions in dist/. +// Every check is an assertion: a failing probe fails the process, so CI gates on it. const fs = require('fs'); const path = require('path'); +// Prints "