@celilo/cli 4.1.0 → 5.0.0

package/src/services/infrastructure-variable-resolver.test.ts

@@ -59,7 +59,7 @@ describe('resolveInfrastructureVariables - Machine Infrastructure', () => {
 
     // Create manifest with infrastructure variables
     const manifest: ModuleManifest = {
-      celilo_contract: '1.0',
+      celilo_contract: '2.0',
       id: moduleId,
       name: 'DNS External',
       version: '1.0.0',
@@ -137,7 +137,7 @@ describe('resolveInfrastructureVariables - Machine Infrastructure', () => {
     const moduleId = 'test-module';
 
     const manifest: ModuleManifest = {
-      celilo_contract: '1.0',
+      celilo_contract: '2.0',
       id: moduleId,
       name: 'Test',
       version: '1.0.0',
@@ -203,7 +203,7 @@ describe('resolveInfrastructureVariables - Proxmox Container Service', () => {
     const moduleId = 'homebridge';
 
     const manifest: ModuleManifest = {
-      celilo_contract: '1.0',
+      celilo_contract: '2.0',
       id: moduleId,
       name: 'Homebridge',
       version: '1.0.0',
@@ -287,7 +287,7 @@ describe('resolveInfrastructureVariables - Proxmox Container Service', () => {
     const moduleId = 'test-module';
 
     const manifest: ModuleManifest = {
-      celilo_contract: '1.0',
+      celilo_contract: '2.0',
       id: moduleId,
       name: 'Test',
       version: '1.0.0',
@@ -350,7 +350,7 @@ describe('resolveInfrastructureVariables - Digital Ocean Container Service', ()
     const moduleId = 'dns-external';
 
     const manifest: ModuleManifest = {
-      celilo_contract: '1.0',
+      celilo_contract: '2.0',
       id: moduleId,
       name: 'DNS External',
       version: '1.0.0',
@@ -436,7 +436,7 @@ describe('resolveInfrastructureVariables - User Override', () => {
     const moduleId = 'dns-external';
 
     const manifest: ModuleManifest = {
-      celilo_contract: '1.0',
+      celilo_contract: '2.0',
       id: moduleId,
       name: 'DNS External',
       version: '1.0.0',
@@ -494,7 +494,7 @@ describe('resolveInfrastructureVariables - Required vs Optional', () => {
     const moduleId = 'test-module';
 
     const manifest: ModuleManifest = {
-      celilo_contract: '1.0',
+      celilo_contract: '2.0',
       id: moduleId,
       name: 'Test',
       version: '1.0.0',
@@ -544,7 +544,7 @@ describe('resolveInfrastructureVariables - Required vs Optional', () => {
     const moduleId = 'test-module';
 
     const manifest: ModuleManifest = {
-      celilo_contract: '1.0',
+      celilo_contract: '2.0',
       id: moduleId,
       name: 'Test',
       version: '1.0.0',
@@ -605,7 +605,7 @@ describe('resolveInfrastructureVariables - Required vs Optional', () => {
 describe('resolveInfrastructureVariables - Edge Cases', () => {
   test('returns empty when no infrastructure variables declared', async () => {
     const manifest: ModuleManifest = {
-      celilo_contract: '1.0',
+      celilo_contract: '2.0',
       id: 'test-module',
       name: 'Test',
       version: '1.0.0',
@@ -632,7 +632,7 @@ describe('resolveInfrastructureVariables - Edge Cases', () => {
 
   test('throws when no infrastructure selected', async () => {
     const manifest: ModuleManifest = {
-      celilo_contract: '1.0',
+      celilo_contract: '2.0',
       id: 'test-module',
       name: 'Test',
       version: '1.0.0',
@@ -660,7 +660,7 @@ describe('resolveInfrastructureVariables - Edge Cases', () => {
     const moduleId = 'test-module';
 
     const manifest: ModuleManifest = {
-      celilo_contract: '1.0',
+      celilo_contract: '2.0',
       id: moduleId,
       name: 'Test',
       version: '1.0.0',

package/src/services/instance-ops.test.ts

@@ -15,7 +15,7 @@ import { deriveInstanceModuleId } from './module-instances';
 
 const manifest = (over: Record<string, unknown> = {}): ModuleManifest =>
   ({
-    celilo_contract: '1.0',
+    celilo_contract: '2.0',
     id: 'lab',
     name: 'BYOI Lab',
     version: '1.0.0',
@@ -110,7 +110,7 @@ describe('createInstanceOps against a real database', () => {
   let parentPath: string;
 
   const SUBMODULE_YAML = `
-celilo_contract: "1.0"
+celilo_contract: "2.0"
 id: lab
 name: BYOI Lab
 version: 1.0.0

package/src/services/module-deploy.dns-repoint.test.ts

@@ -64,7 +64,7 @@ describe('repointDnsPrimaryForProvider (module-orchestrator-primitives slice 3)'
         id: MODULE_ID,
         name: MODULE_ID,
         version: '1.3.4',
-        manifestData: { id: MODULE_ID, name: MODULE_ID, version: '1.3.4', celilo_contract: '1.0' },
+        manifestData: { id: MODULE_ID, name: MODULE_ID, version: '1.3.4', celilo_contract: '2.0' },
         sourcePath: `/tmp/${MODULE_ID}`,
       })
       .run();

package/src/services/module-pause-quiescence.test.ts

@@ -36,7 +36,7 @@ function manifestWithHooks(id: string) {
     id,
     name: id,
     version: '1.0.0',
-    celilo_contract: '1.0',
+    celilo_contract: '2.0',
     provides: { capabilities: [] },
     requires: { capabilities: [] },
     hooks: {

package/src/services/module-pause.test.ts

@@ -36,7 +36,7 @@ function makeManifest(
     id,
     name: id,
     version: '1.0.0',
-    celilo_contract: '1.0',
+    celilo_contract: '2.0',
     provides: {
       capabilities: (opts.provides ?? []).map((name) => ({
         name,

package/src/services/module-subscriptions.test.ts

@@ -19,7 +19,7 @@ import {
 } from './module-subscriptions';
 
 const baseManifest = (overrides: Partial<ModuleManifest> = {}): ModuleManifest => ({
-  celilo_contract: '1.0',
+  celilo_contract: '2.0',
   id: 'lunacycle',
   name: 'Lunacycle',
   version: '1.0.0',
@@ -357,7 +357,7 @@ describe('resyncAllSubscriptions (ISS-0088)', () => {
     subs: Array<{ name: string; pattern: string; handler: string }>,
   ): void {
     const manifest = JSON.stringify({
-      celilo_contract: '1.0',
+      celilo_contract: '2.0',
       id,
       name: id,
       version: '1.0.0',

package/src/services/module-types-generator.test.ts

@@ -15,7 +15,7 @@ import {
 
 function baseManifest(overrides: Partial<ModuleManifest> = {}): ModuleManifest {
   return {
-    celilo_contract: '1.0',
+    celilo_contract: '2.0',
     id: 'test-module',
     name: 'Test Module',
     version: '1.0.0',

package/src/services/module-validator/contract-version.test.ts

@@ -3,7 +3,7 @@ import { checkContractVersion } from './contract-version';
 
 describe('checkContractVersion', () => {
   test('ok when manifest declares the latest supported contract', () => {
-    const r = checkContractVersion({ celilo_contract: '1.0' });
+    const r = checkContractVersion({ celilo_contract: '2.0' });
     expect(r.status).toBe('ok');
     expect(r.message).toContain('latest');
   });
@@ -12,13 +12,13 @@ describe('checkContractVersion', () => {
     const r = checkContractVersion({});
     expect(r.status).toBe('fail');
     expect(r.message).toContain('missing');
-    expect(r.suggestedValue).toBe('1.0');
+    expect(r.suggestedValue).toBe('2.0');
   });
 
   test('fail when celilo_contract claims an unknown version', () => {
     const r = checkContractVersion({ celilo_contract: '99.0' });
     expect(r.status).toBe('fail');
     expect(r.message).toContain('unknown');
-    expect(r.suggestedValue).toBe('1.0');
+    expect(r.suggestedValue).toBe('2.0');
   });
 });

package/src/services/network-ensure.test.ts

@@ -16,7 +16,7 @@ import { type NetworkAsker, ensureRequiredNetworks } from './network-ensure';
 
 function manifestRequiring(...networks: string[]): ModuleManifest {
   return {
-    celilo_contract: '1.0',
+    celilo_contract: '2.0',
     id: 'test-module',
     name: 'Test Module',
     version: '0.1.0',

package/src/services/proxmox-reconcile.test.ts

@@ -44,7 +44,7 @@ function seedProviderModule(opts: { configs?: Record<string, string> } = {}) {
         id: PROVIDER_MODULE_ID,
         name: PROVIDER_MODULE_ID,
         version: '1.0.0',
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
       },
       sourcePath: `/tmp/${PROVIDER_MODULE_ID}`,
     })
@@ -107,7 +107,7 @@ function seedContainerLxc(opts: {
         id: opts.moduleId,
         name: opts.moduleId,
         version: '1.0.0',
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
       },
       sourcePath: `/tmp/${opts.moduleId}`,
     })

package/src/services/remove-guard.test.ts

@@ -14,7 +14,7 @@ function manifest(
     id,
     name: id,
     version: '1.0.0',
-    celilo_contract: '1.0',
+    celilo_contract: '2.0',
     provides: { capabilities: [] },
     requires: { capabilities: (opts.requires ?? []).map((name) => ({ name, version: '1.0.0' })) },
     optional: opts.optional

package/src/services/update/dep-graph.test.ts

@@ -20,7 +20,7 @@ function makeManifest(
     id,
     name: id,
     version: '1.0.0',
-    celilo_contract: '1.0',
+    celilo_contract: '2.0',
     provides: opts.provides
       ? {
           capabilities: opts.provides.map((name) => ({

package/src/services/update/orchestrator.test.ts

@@ -24,7 +24,7 @@ function makeManifest(
     id,
     name: id,
     version: '1.0.0',
-    celilo_contract: '1.0',
+    celilo_contract: '2.0',
     provides: opts.provides
       ? {
           capabilities: opts.provides.map((name) => ({

package/src/variables/capability-self-ref.test.ts

@@ -261,7 +261,7 @@ describe('Capability data $self: variable resolution', () => {
       id: 'lunacycle',
       name: 'LunaCycle',
       version: '1.0.0',
-      celilo_contract: '1.0',
+      celilo_contract: '2.0',
       variables: {
         owns: [
           {
@@ -388,7 +388,7 @@ describe('Capability data $self: variable resolution', () => {
       id: 'lunacycle',
       name: 'LunaCycle',
       version: '1.0.0',
-      celilo_contract: '1.0',
+      celilo_contract: '2.0',
       variables: {
         owns: [
           {

package/src/variables/computed/computed-integration.test.ts

@@ -29,7 +29,7 @@ let db: DbClient;
 /** Fresh manifest per call — never share a mutable manifest across tests. */
 function namecheapManifest(computedValue = 'keys(secret.ddns_passwords)'): ModuleManifest {
   return {
-    celilo_contract: '1.0',
+    celilo_contract: '2.0',
     id: 'namecheap',
     name: 'Namecheap',
     version: '4.1.0',

package/src/variables/declarative-derivation.test.ts

@@ -441,7 +441,7 @@ describe('applyDeclarativeDerivations', () => {
   describe('basic derivations', () => {
     test('applies single derivation', () => {
       const manifest: ModuleManifest = {
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
         id: 'test-module',
         name: 'Test Module',
         version: '1.0.0',
@@ -477,7 +477,7 @@ describe('applyDeclarativeDerivations', () => {
 
     test('applies multiple derivations in order', () => {
       const manifest: ModuleManifest = {
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
         id: 'test-module',
         name: 'Test Module',
         version: '1.0.0',
@@ -523,7 +523,7 @@ describe('applyDeclarativeDerivations', () => {
   describe('user config precedence', () => {
     test('does not overwrite user-provided values', () => {
       const manifest: ModuleManifest = {
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
         id: 'test-module',
         name: 'Test Module',
         version: '1.0.0',
@@ -562,7 +562,7 @@ describe('applyDeclarativeDerivations', () => {
   describe('dependency resolution', () => {
     test('resolves dependencies in declaration order', () => {
       const manifest: ModuleManifest = {
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
         id: 'test-module',
         name: 'Test Module',
         version: '1.0.0',
@@ -619,7 +619,7 @@ describe('applyDeclarativeDerivations', () => {
   describe('error handling', () => {
     test('throws error for required variable with missing dependency', () => {
       const manifest: ModuleManifest = {
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
         id: 'test-module',
         name: 'Test Module',
         version: '1.0.0',
@@ -655,7 +655,7 @@ describe('applyDeclarativeDerivations', () => {
 
     test('silently skips optional variable with missing dependency', () => {
       const manifest: ModuleManifest = {
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
         id: 'test-module',
         name: 'Test Module',
         version: '1.0.0',
@@ -693,7 +693,7 @@ describe('applyDeclarativeDerivations', () => {
 
     test('derives scalar numeric values from capability data', () => {
       const manifest: ModuleManifest = {
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
         id: 'test-module',
         name: 'Test Module',
         version: '1.0.0',
@@ -731,7 +731,7 @@ describe('applyDeclarativeDerivations', () => {
 
     test('continues to skip structured derivations', () => {
       const manifest: ModuleManifest = {
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
         id: 'test-module',
         name: 'Test Module',
         version: '1.0.0',
@@ -776,7 +776,7 @@ describe('applyDeclarativeDerivations', () => {
     // context. The key must remain unset rather than being poisoned with the raw template.
     test('does not set selfConfig when capability value still contains $self: ref', () => {
       const manifest: ModuleManifest = {
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
         id: 'lunacycle',
         name: 'Lunacycle',
         version: '1.0.0',
@@ -816,7 +816,7 @@ describe('applyDeclarativeDerivations', () => {
 
     test('uses existing selfConfig value when capability returns unresolved $self: ref', () => {
       const manifest: ModuleManifest = {
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
         id: 'lunacycle',
         name: 'Lunacycle',
         version: '1.0.0',
@@ -860,7 +860,7 @@ describe('applyDeclarativeDerivations', () => {
     // all 5 resolution passes unchanged and must be skipped.
     test('does not set selfConfig when capability value contains $secret: ref', () => {
       const manifest: ModuleManifest = {
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
         id: 'consumer',
         name: 'Consumer',
         version: '1.0.0',
@@ -903,7 +903,7 @@ describe('applyDeclarativeDerivations', () => {
     // end-to-end, not just resolveDeclarativeDerivation in isolation.
     test('resolves $system: ref inside capability value', () => {
       const manifest: ModuleManifest = {
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
         id: 'caddy',
         name: 'Caddy',
         version: '1.0.0',
@@ -946,7 +946,7 @@ describe('applyDeclarativeDerivations', () => {
     // on first pass (the other isn't set yet). Both should remain unset.
     test('silently skips both optional variables in a circular reference', () => {
       const manifest: ModuleManifest = {
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
         id: 'test-module',
         name: 'Test Module',
         version: '1.0.0',
@@ -990,7 +990,7 @@ describe('applyDeclarativeDerivations', () => {
 
     test('throws when either circular variable is required', () => {
       const manifest: ModuleManifest = {
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
         id: 'test-module',
         name: 'Test Module',
         version: '1.0.0',
@@ -1040,7 +1040,7 @@ describe('applyDeclarativeDerivations', () => {
     // Equivalent to the {var} syntax; reads from selfConfig.
     test('resolves $self: against selfConfig like {var}', () => {
       const manifest: ModuleManifest = {
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
         id: 'test-module',
         name: 'Test Module',
         version: '1.0.0',
@@ -1079,7 +1079,7 @@ describe('applyDeclarativeDerivations', () => {
       // auth_url derives from "https://auth.$self:domain" where domain
       // is another self-owned variable set by the user.
       const manifest: ModuleManifest = {
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
         id: 'authentik',
         name: 'Authentik',
         version: '1.0.0',
@@ -1121,7 +1121,7 @@ describe('applyDeclarativeDerivations', () => {
 
     test('throws when $self: references an undeclared variable', () => {
       const manifest: ModuleManifest = {
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
         id: 'test',
         name: 'Test',
         version: '1.0.0',
@@ -1162,7 +1162,7 @@ describe('applyDeclarativeDerivations', () => {
   describe('skips', () => {
     test('skips variables without derive_from', () => {
       const manifest: ModuleManifest = {
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
         id: 'test-module',
         name: 'Test Module',
         version: '1.0.0',
@@ -1199,7 +1199,7 @@ describe('applyDeclarativeDerivations', () => {
 
     test('handles manifest with no variables', () => {
       const manifest: ModuleManifest = {
-        celilo_contract: '1.0',
+        celilo_contract: '2.0',
         id: 'test-module',
         name: 'Test Module',
         version: '1.0.0',