@myelinbridge/cli 0.10.0 → 0.11.1

package/README.md

@@ -13,6 +13,8 @@ npx @myelinbridge/cli ping                 # verifies auth, prints your projects
 npx @myelinbridge/cli datasets             # what you can deliver to, and whose move it is
 npx @myelinbridge/cli contract --dataset onco1-wes            # what is expected of your delivery
 npx @myelinbridge/cli sample-depth 1 --dataset onco1-wes      # once, before your first submit
+npx @myelinbridge/cli roles --dataset onco1-wes               # which file plays which role
+npx @myelinbridge/cli roles set samplesheet /metadata/samplesheet.csv --dataset onco1-wes
 npx @myelinbridge/cli check ./run_042 --dataset onco1-wes    # validate BEFORE uploading a byte
 npx @myelinbridge/cli push  ./run_042 --dataset onco1-wes --submit
 ```
@@ -27,6 +29,31 @@ npx @myelinbridge/cli push  ./run_042 --dataset onco1-wes --submit
   deliveries — after that, re-asserting the current value still succeeds and only a
   *change* exits `2`. This is the only dataset field you can write.
 
+- **Declare which file plays which role — whenever you like (0.11.0).** Quality
+  checks read files by ROLE (`samplesheet`, `checksum_manifest`, `qc_report`,
+  `subject_roster`, `capture_bed`), not by a filing convention you never agreed
+  to. `roles` shows what is declared and what Myelin detected in your
+  deliveries; `roles set <role> <path>` confirms a file; `roles none <role>`
+  states your dataset has no such file (a statement, not a gap). Nothing is
+  ever required to deliver — an undeclared role just means the checks that read
+  it report *"could not check"* instead of running, so declaring is how you
+  activate more checks before upload and catch problems before review does.
+  Unlike `sample-depth` it **never locks**, and it is idempotent, so a pipeline
+  can assert it on every run. The samplesheet is **sticky**: send it once and
+  every later delivery reconciles against it — a partial delivery (93 of 96) is
+  reported as a state, never as a failure.
+
+- **A missing path in `roles set` is refused, not guessed (0.11.1).** Flags are
+  not positional arguments: `roles set samplesheet --dataset onco1-wes` (path
+  omitted) used to declare the samplesheet at the literal path `--dataset` and
+  print a tick. Every samplesheet check then reported *"could not check — the
+  declared file is not in the delivery"* on a delivery that was fine. The
+  command now exits with its usage line instead.
+
+- **`--dataset` takes the slug in any case (0.11.0).** Slugs are matched
+  case-insensitively (`onco1-wes` and `ONCO1-WES` name the same dataset); an id
+  or an exact name works too.
+
 - **Resume = re-run.** `push` is idempotent: already-uploaded files are skipped
   (path + size), and within a large file, parts that already landed are skipped
   too (S3 multipart). Uploads go direct to storage over short-lived presigned

package/bin/myelin.js

@@ -191,9 +191,15 @@ async function resolveDataset(ref) {
     die(`Dataset ${ref} not found in this key's scope`)
   }
   const me = expectOk(await api('GET', '/me'), 'auth')
+  // Slugs are matched case-insensitively: seeded datasets carry lowercase slugs
+  // while the portal generates uppercase ones, and the slug is a handle a human
+  // types — the case of the fleet should never be their problem.
+  const want = ref.toLowerCase()
   for (const p of me.projects) {
     const r = expectOk(await api('GET', `/projects/${p.id}/datasets`), 'datasets')
-    const hit = r.datasets.find((d) => d.slug === ref || d.name === ref)
+    const hit = r.datasets.find(
+      (d) => (typeof d.slug === 'string' && d.slug.toLowerCase() === want) || d.name === ref,
+    )
     if (hit) return hit
   }
   die(`No dataset with slug or name "${ref}" in this key's scope`)
@@ -896,6 +902,69 @@ async function cmdSampleDepth() {
   }
 }
 
+// Which delivered file plays each role (sprint 19). Checks read files by ROLE
+// — declaring one activates every check that was waiting on it, and nothing
+// here is ever required to deliver. Unlike sample-depth this never locks:
+// declarations apply forward, and re-declaring is allowed for the life of the
+// dataset (idempotent — assert it on every run).
+async function cmdRoles() {
+  const dsRef = opt('dataset')
+  if (!dsRef) die('Usage: myelin roles [set <role> <path> | none <role> | clear <role>] --dataset <slug|id>')
+  const dataset = await resolveDataset(dsRef)
+  const sub = args[1]
+
+  if (sub === 'set' || sub === 'none' || sub === 'clear') {
+    const role = args[2]
+    if (!role) die(`Usage: myelin roles ${sub} <role>${sub === 'set' ? ' <path>' : ''} --dataset <slug|id>`)
+    const body = { role }
+    if (sub === 'set') {
+      const path = args[3]
+      // ⚠ `args` only strips --json, so an omitted path lets the NEXT FLAG slide
+      // into its place: `roles set samplesheet --dataset onco1-wes` used to
+      // declare the samplesheet at `/--dataset`. The API accepts it (printable,
+      // under 255 chars), the CLI printed a tick, and every samplesheet check
+      // then answered role_file_missing on a delivery that was fine.
+      if (!path || path.startsWith('--')) {
+        die('Usage: myelin roles set <role> <path> --dataset <slug|id>')
+      }
+      body.path = path
+    } else if (sub === 'none') {
+      body.none = true
+    } else {
+      body.clear = true
+    }
+    const j = expectOk(await api('PUT', `/datasets/${dataset.id}/file-roles`, body), 'file-roles')
+    emit(j)
+    if (!JSON_MODE) {
+      out(
+        j.changed
+          ? `✓ ${role} ${sub === 'set' ? `→ ${body.path}` : sub === 'none' ? 'declared: no such file in this dataset' : 'declaration cleared'} for ${dataset.name}.`
+          : `${role} already declared that way for ${dataset.name} — nothing to change.`,
+      )
+    }
+    return
+  }
+
+  const j = expectOk(await api('GET', `/datasets/${dataset.id}/file-roles`), 'file-roles')
+  emit(j)
+  if (JSON_MODE) return
+  out(`${dataset.name} — delivery file roles`)
+  const declaredBy = new Map(j.declarations.map((d) => [d.role, d]))
+  for (const role of ['samplesheet', 'checksum_manifest', 'qc_report', 'subject_roster', 'capture_bed']) {
+    const d = declaredBy.get(role)
+    if (d) {
+      out(`  ${role.padEnd(18)} ${d.declared_none ? '(no such file in this dataset — declared)' : d.path}`)
+    } else {
+      const cands = j.candidates.filter((c) => c.role === role)
+      out(`  ${role.padEnd(18)} not declared${cands.length ? ` — looks like: ${cands.map((c) => c.path).join(', ')}` : ''}`)
+    }
+  }
+  const undeclared = 5 - j.declarations.length
+  if (undeclared > 0) {
+    out(`\nDeclaring a role activates the checks that read it — run: myelin roles set <role> <path> --dataset ${dsRef}`)
+  }
+}
+
 function cmdHelp() {
   console.log(`myelin — Partner Ingestion CLI
 
@@ -910,6 +979,8 @@ Commands:
   projects                                list scoped projects
   datasets                                list datasets with "your move" hints
   sample-depth <0-5> --dataset <slug|id>  declare the folder depth a sample sits at (locks after 1st submit)
+  roles [set|none|clear …] --dataset <…>  which file plays each role (samplesheet, checksum list, …) —
+                                          declaring one activates the checks that read it; never locks
   contract --dataset <slug|id>            what this dataset expects of your delivery
   check <dir> --dataset <slug|id>         preflight local files against quality rules (no upload)
   push <dir> --dataset <slug|id>          create/resume a delivery and upload (resumable; re-run to resume)
@@ -1008,6 +1079,7 @@ const commands = {
   projects: cmdProjects,
   datasets: cmdDatasets,
   'sample-depth': cmdSampleDepth,
+  roles: cmdRoles,
   contract: cmdContract,
   check: cmdCheck,
   push: cmdPush,

package/package.json

@@ -1,23 +1,23 @@
-{
-  "name": "@myelinbridge/cli",
-  "version": "0.10.0",
-  "description": "Myelin Partner Ingestion CLI — push R&D data deliveries from a pipeline: preflight against the client's quality rules, resumable upload, submit, track review outcomes.",
-  "type": "module",
-  "bin": {
-    "myelin": "bin/myelin.js"
-  },
-  "files": [
-    "bin/",
-    "README.md"
-  ],
-  "engines": {
-    "node": ">=20"
-  },
-  "keywords": [
-    "myelin",
-    "pharma",
-    "ingestion",
-    "s3"
-  ],
-  "license": "UNLICENSED"
-}
+{
+  "name": "@myelinbridge/cli",
+  "version": "0.11.1",
+  "description": "Myelin Partner Ingestion CLI — push R&D data deliveries from a pipeline: preflight against the client's quality rules, resumable upload, submit, track review outcomes.",
+  "type": "module",
+  "bin": {
+    "myelin": "bin/myelin.js"
+  },
+  "files": [
+    "bin/",
+    "README.md"
+  ],
+  "engines": {
+    "node": ">=20"
+  },
+  "keywords": [
+    "myelin",
+    "pharma",
+    "ingestion",
+    "s3"
+  ],
+  "license": "UNLICENSED"
+}