breakaway 2.0.0-main.48 → 2.0.0-main.49

package/npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
   "name": "breakaway",
-  "version": "2.0.0-main.48",
+  "version": "2.0.0-main.49",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "breakaway",
-      "version": "2.0.0-main.48",
+      "version": "2.0.0-main.49",
       "license": "FSL-1.1-Apache-2.0",
       "bin": {
         "breakaway": "scripts/tasks.mjs"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "breakaway",
-  "version": "2.0.0-main.48",
+  "version": "2.0.0-main.49",
   "description": "The task board for you and your coding agents: a Cloudflare Worker, its web app, Taskwarrior sync, and the CLI (npx breakaway).",
   "license": "FSL-1.1-Apache-2.0",
   "type": "module",

package/src/infra-cloudflare-plan.js

@@ -25,6 +25,7 @@ import {
   API,
   BINDING_TARGETS,
   CloudflareError,
+  MANAGED,
   PRICES,
   accountOf,
   bindingTarget,
@@ -44,21 +45,8 @@ import {
 /** @typedef {import('./infra-provider.js').ApplyResult} ApplyResult */
 /** @typedef {import('./infra-cloudflare.js').LiveAccount} LiveAccount */
 
-/**
- * The settings Architect manages, by kind. Everything else a resource carries (a Worker's handlers, secrets, and live
- * versions, a database's size, a container's instances) is Cloudflare's or the deploy's, and is never compared.
- */
-export const MANAGED = {
-  worker: ['compatibilityDate', 'compatibilityFlags', 'usageModel', 'observability', 'placement', 'bindings', 'crons'],
-  'durable-object': [],
-  d1: [],
-  kv: [],
-  r2: ['cors', 'lifecycle'],
-  queue: ['deliveryDelay', 'deliveryPaused', 'retention'],
-  container: ['maxInstances'],
-  route: ['worker'],
-  'custom-domain': ['worker', 'environment'],
-};
+/** The settings Architect manages, by kind: infra-cloudflare.js keeps them, on each kind. */
+export { MANAGED };
 
 /** Kinds Cloudflare names by their name, so a rename is a change of its own (KV's title, a route's pattern). */
 const RENAMES = new Set(['kv', 'route']);

package/src/infra-cloudflare.js

@@ -32,12 +32,25 @@ import { apply, estimate, plan } from './infra-cloudflare-plan.js';
 export const API = 'https://api.cloudflare.com/client/v4';
 
 const BASE = ['create', 'update', 'delete'];
+/**
+ * The settings Architect manages, by kind. Everything else a resource carries (a Worker's handlers, secrets, and live
+ * versions, a database's size, a container's instances) is Cloudflare's or the deploy's, and is never compared.
+ */
+export const MANAGED = {
+  worker: ['compatibilityDate', 'compatibilityFlags', 'usageModel', 'observability', 'placement', 'bindings', 'crons'],
+  'durable-object': [],
+  d1: [],
+  kv: [],
+  r2: ['cors', 'lifecycle'],
+  queue: ['deliveryDelay', 'deliveryPaused', 'retention'],
+  container: ['maxInstances'],
+  route: ['worker'],
+  'custom-domain': ['worker', 'environment'],
+};
+
 /** BRK-227 adds `scale` to queues and `scale` and `restart` to containers. */
 export const CLOUDFLARE_KINDS = Object.fromEntries(
-  ['worker', 'durable-object', 'd1', 'kv', 'r2', 'queue', 'container', 'route', 'custom-domain'].map((k) => [
-    k,
-    { changes: [...BASE] },
-  ]),
+  Object.entries(MANAGED).map(([k, settings]) => [k, { changes: [...BASE], settings: [...settings] }]),
 );
 
 /**

package/src/infra-provider.js

@@ -147,6 +147,9 @@ export const COST_NOTE_MAX = 500;
  *   owner for any change to one (BRK-181)
  * @property {string[]} [accessSettings] settings of the kind that decide who or what can reach it (like `public`): policy
  *   asks the owner for a change to one
+ * @property {string[]} [settings] the settings a desired state sets for the kind; the rest of a resource's attrs are what
+ *   the platform reports by itself (versions, sizes, deploys), which a draft of the desired state leaves out (BRK-240).
+ *   Without it, every attr is a setting.
  * @property {string} [scales] the whole-number setting a `scale` changes (like `instances`): required when the kind
  *   declares `scale`, so an envelope (BRK-186) knows what its bounds are bounds of
  */
@@ -241,6 +244,8 @@ export function checkProvider(provider) {
       fail(what, `${kind}'s access is not true or false`);
     if (spec.accessSettings !== undefined && (!Array.isArray(spec.accessSettings) || !spec.accessSettings.every(text)))
       fail(what, `${kind}'s accessSettings is not a list of settings' names`);
+    if (spec.settings !== undefined && (!Array.isArray(spec.settings) || !spec.settings.every(text)))
+      fail(what, `${kind}'s settings is not a list of settings' names`);
     if (spec.changes.includes('scale') ? !text(spec.scales) : spec.scales !== undefined)
       fail(what, `${kind}'s scales names the setting a scale changes, and only a kind that scales has one`);
   }