start-command 0.39.1 → 0.40.0

package/test/execution-store.js

@@ -367,13 +367,12 @@ describe('ExecutionStore', () => {
     const linoPath = path.join(TEST_APP_FOLDER, LINO_DB_FILE);
     expect(fs.existsSync(linoPath)).toBe(true);
 
-    // Read and verify content - lino format uses base64 encoding for strings
+    // lino-objects-codec 0.9 writes readable links notation (#253)
     const content = fs.readFileSync(linoPath, 'utf8');
     expect(content.length).toBeGreaterThan(0);
-    // Verify content has array structure
-    expect(content).toContain('(array');
-    // Verify PID is stored (as int)
-    expect(content).toContain('(int 12345)');
+    expect(content).toContain('command "echo hello"');
+    // Verify PID is stored as a bare integer
+    expect(content).toContain('pid 12345');
   });
 });
 
@@ -449,6 +448,51 @@ describe('ExecutionStore with lino-objects-codec', () => {
     expect(retrieved.command).toBe('echo "Hello 世界 🌍"');
     expect(retrieved.logPath).toBe('/tmp/unicode-日本語.log');
   });
+
+  it('reads a store written by lino-objects-codec 0.4.0 and rewrites it readable (#253)', () => {
+    fs.mkdirSync(TEST_APP_FOLDER, { recursive: true });
+    fs.copyFileSync(
+      path.join(__dirname, 'fixtures/executions-lino-objects-codec-0.4.0.lino'),
+      path.join(TEST_APP_FOLDER, LINO_DB_FILE)
+    );
+    const uuid = '25300000-0000-4000-8000-000000000253';
+
+    const legacy = store.get(uuid);
+    expect(legacy.pid).toBe(4242);
+    expect(legacy.exitCode).toBe(0);
+    expect(legacy.command).toBe('echo "legacy store"');
+
+    store.save(new ExecutionRecord({ command: 'echo next' }));
+    const content = fs.readFileSync(
+      path.join(TEST_APP_FOLDER, LINO_DB_FILE),
+      'utf8'
+    );
+    expect(content).toContain('command "echo next"');
+    expect(content).not.toContain('(str ');
+    expect(store.get(uuid).command).toBe('echo "legacy store"');
+  });
+
+  it('reads a readable store written by lino-objects-codec 0.9 (#253)', () => {
+    // rust/tests/regression_253.rs reads the same fixture
+    fs.mkdirSync(TEST_APP_FOLDER, { recursive: true });
+    fs.copyFileSync(
+      path.join(__dirname, 'fixtures/executions-lino-objects-codec-0.9.3.lino'),
+      path.join(TEST_APP_FOLDER, LINO_DB_FILE)
+    );
+
+    const record = store.get('25300000-0000-4000-8000-000000000930');
+    expect(record.pid).toBe(4242);
+    expect(record.exitCode).toBe(0);
+    expect(record.command).toBe(
+      `echo "readable store" (with) \\ odd chars 'q' 世界`
+    );
+    expect(record.endTime).toBe('2026-10-01T00:00:01.000Z');
+    expect(record.options).toEqual({
+      isolated: 'docker',
+      keepAlive: false,
+      nested: { list: [1, 'two', null] },
+    });
+  });
 });
 
 describe('ExecutionStore verifyConsistency', () => {

package/test/fixtures/executions-lino-objects-codec-0.4.0.lino

@@ -0,0 +1 @@
+(array (object ((str dXVpZA==) (str MjUzMDAwMDAtMDAwMC00MDAwLTgwMDAtMDAwMDAwMDAwMjUz)) ((str cGlk) (int 4242)) ((str c3RhdHVz) (str ZXhlY3V0ZWQ=)) ((str ZXhpdENvZGU=) (int 0)) ((str Y29tbWFuZA==) (str ZWNobyAibGVnYWN5IHN0b3JlIg==)) ((str bG9nUGF0aA==) (str L3RtcC9zdGFydC0yNTMubG9n)) ((str c3RhcnRUaW1l) (str MjAyNi0xMC0wMVQwMDowMDowMC4wMDBa)) ((str ZW5kVGltZQ==) (str MjAyNi0xMC0wMVQwMDowMDowMS4wMDBa)) ((str d29ya2luZ0RpcmVjdG9yeQ==) (str L3RtcA==)) ((str c2hlbGw=) (str L2Jpbi9zaA==)) ((str cGxhdGZvcm0=) (str bGludXg=)) ((str b3B0aW9ucw==) (object ((str aXNvbGF0ZWQ=) (null))))))

package/test/fixtures/executions-lino-objects-codec-0.9.3.lino

@@ -0,0 +1,26 @@
+(
+  (
+    uuid "25300000-0000-4000-8000-000000000930"
+    pid 4242
+    status "executed"
+    exitCode 0
+    command """echo "readable store" (with) \ odd chars 'q' 世界"""
+    logPath "/tmp/start-253-readable.log"
+    startTime "2026-10-01T00:00:00.000Z"
+    endTime "2026-10-01T00:00:01.000Z"
+    workingDirectory "/tmp"
+    shell "/bin/sh"
+    platform "linux"
+    options (
+      isolated "docker"
+      keepAlive false
+      nested (
+        list (
+          1
+          "two"
+          null
+        )
+      )
+    )
+  )
+)

package/test/javascript-first-ci.js

@@ -233,6 +233,35 @@ describe('JavaScript first CI gate', () => {
     );
   });
 
+  it('treats generated Rust regenerated by a translator bump as no change', () => {
+    const features = [
+      {
+        id: 'generated',
+        javascript: { implementation: ['js/source.mjs'], tests: [] },
+        rust: { implementation: ['rust/generated.rs'], tests: [] },
+      },
+    ];
+    const generated = [
+      { javascript: 'js/source.mjs', rust: 'rust/generated.rs' },
+    ];
+    assert.deepEqual(
+      checkFeatureChanges(features, ['rust/generated.rs'], generated),
+      []
+    );
+    assert.deepEqual(
+      checkFeatureChanges(
+        features,
+        ['js/source.mjs', 'rust/generated.rs'],
+        generated
+      ),
+      []
+    );
+    assert.equal(
+      checkFeatureChanges(features, ['rust/generated.rs']).length,
+      1
+    );
+  });
+
   it('keeps both halves of a regression test under one feature', () => {
     const feature = (id, js, rust) => ({
       id,