From 167de94fe4ef82239d841671beab76d301928f33 Mon Sep 17 00:00:00 2001 From: Christopher Monsanto Date: Mon, 17 Aug 2026 15:41:09 -0400 Subject: [PATCH] Drop the last vestiges of the __key convention The upload command names the asset set on its command line now. Co-Authored-By: Claude Fable 5 --- tools/deploy/api.ts | 3 +-- tools/deploy/test/api.test.ts | 4 ++-- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/tools/deploy/api.ts b/tools/deploy/api.ts index d3d7e76c..238c1c6a 100644 --- a/tools/deploy/api.ts +++ b/tools/deploy/api.ts @@ -18,8 +18,7 @@ export interface SrcFile extends pathlib.Path { export type CopySource = Artifact | SrcFile | string; // string = repo-relative path // The finish API: nice naming over built artifacts and raw sources. Ops are -// queued in call order (the tar entry order), so __key-style entries must be -// written first. +// queued in call order, which is the tar entry order. export interface DeployCtx { copy(src : CopySource, dst : string) : void; write(dst : string, data : string) : void; diff --git a/tools/deploy/test/api.test.ts b/tools/deploy/test/api.test.ts index 4fe8b5c5..74821456 100644 --- a/tools/deploy/test/api.test.ts +++ b/tools/deploy/test/api.test.ts @@ -65,11 +65,11 @@ test('ctx queues artifact copies from the CAS, writes and reads', () => { const artifact = makeArtifact(casDir, 'bytes', 'webp'); const aq = new ActionQueue(); const ctx = makeCtx(casDir, aq); - ctx.write('__key', 'sprites'); + ctx.write('m.json', '{}'); ctx.copy(artifact, 'sprites/x.webp'); assert.equal(ctx.read(artifact), 'bytes'); const ops = aq.log.filter(e => e.type === 'Op'); - assert.deepEqual(ops.map(e => e.dst), ['__key', 'sprites/x.webp']); + assert.deepEqual(ops.map(e => e.dst), ['m.json', 'sprites/x.webp']); assert.equal((ops[1] as {op : {src : string}}).op.src, casPath(casDir, artifact.hash, 'webp')); });