@celilo/e2e 0.17.0 → 0.17.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@celilo/e2e",
-  "version": "0.17.0",
+  "version": "0.17.1",
   "description": "E2E test infrastructure for Celilo-deployed applications. Provides a simulated internet with DNS hierarchy, ACME server, firewalls, and target machines in Docker.",
   "type": "module",
   "main": "./src/index.ts",
@@ -37,7 +37,7 @@
     "README.md"
   ],
   "dependencies": {
-    "@celilo/capabilities": "^2.1.0",
+    "@celilo/capabilities": "^2.4.0",
     "@celilo/cli-display": "^0.2.0",
     "@celilo/event-bus": "^0.6.0",
     "@celilo/terraform-fake": "^0.3.1",

package/registry-server/package.json

@@ -8,7 +8,7 @@
     "start": "bun run src/index.ts",
     "dev": "bun --watch src/index.ts",
     "build:binary": "bun build --compile --target=bun-linux-x64 --outfile=dist/celilo-registry-server src/index.ts",
-    "test": "bun test"
+    "test": "bun test --timeout 30000"
   },
   "devDependencies": {
     "bun-types": "latest",

package/scripts/stage-apt-repo.ts

@@ -98,7 +98,9 @@ export function stageAptRepo(repoRoot: string, pkgDir: string): boolean {
 
   const debs = readdirSync(distDir).filter((f) => f.endsWith('.deb') && f.includes(version));
   for (const deb of debs) {
-    cpSync(join(distDir, deb), join(pool, deb));
+    // `force` explicit: a rebuilt .deb at the same version must replace the
+    // one already pooled, or build-infra serves stale bytes.
+    cpSync(join(distDir, deb), join(pool, deb), { force: true });
   }
 
   console.log(

package/src/module-host.ts

@@ -48,7 +48,6 @@ export function parseModuleHost(statusOutput: string): ModuleHost | null {
   // both, and a raw match silently finds nothing.
   const lines = statusOutput
     .split('\n')
-    // biome-ignore lint/suspicious/noControlCharactersInRegex: stripping ANSI is the point
     .map((line) => line.replace(/\x1b\[[0-9;]*m/g, '').replace(/^[\s│|]+/, ''));
 
   for (const line of lines) {

package/src/router-swap.ts

@@ -102,7 +102,21 @@ export function forwardedPorts(listing: string): string[] {
   return [...new Set(ports)].sort();
 }
 
-/** The DHCP pool's advertised DNS servers, read back off the device. */
+/**
+ * The DHCP pool's advertised DNS servers, read back off the device.
+ *
+ * Queries the pool INSTANCE (`…Pool.1.`), not the collection (`…Pool.`). The
+ * simulator answers the collection form with `{"Objects":[]}` — no error, just
+ * an empty list — so the collection form reads as "the router advertises no DNS
+ * servers" whatever is actually configured. Both `greenwave` and `axon` write
+ * through `Device.DHCPv4.Server.Pool.1.`
+ * (`modules/greenwave/scripts/isp-router-functions.ts:153`), so this reads back
+ * exactly where the module wrote.
+ *
+ * Nothing called this until `e2e/tests/provider-arrival-backfill.test.ts`, which
+ * is how the mismatch survived: against the collection form an assertion fails
+ * identically whether the router was configured or not.
+ */
 export async function routerDhcpDnsServers(
   net: NetworkHandle,
   device: RouterDevice,
@@ -115,7 +129,7 @@ export async function routerDhcpDnsServers(
   );
   const listed = await net.exec(
     'fw-isp',
-    `curl -sk 'https://${greenwaveRouterIp()}/cgi/cgi_get?Object=Device.DHCPv4.Server.Pool.' -b /tmp/dhcp-cookies`,
+    `curl -sk 'https://${greenwaveRouterIp()}/cgi/cgi_get?Object=Device.DHCPv4.Server.Pool.1.' -b /tmp/dhcp-cookies`,
   );
   return listed.stdout;
 }