@hasna/skills 0.2.0 → 0.2.1

package/dist/sdk/runs.d.ts

@@ -38,23 +38,7 @@ export declare const runAdmissionSchema: z.ZodObject<{
     skill: z.ZodString;
     status: z.ZodLiteral<"admitted">;
     createdAt: z.ZodString;
-}, "strip", z.ZodTypeAny, {
-    status: "admitted";
-    createdAt: string;
-    skill: string;
-    contractVersion: 1;
-    runId: string;
-    leaseGeneration: number;
-    attemptId: string;
-}, {
-    status: "admitted";
-    createdAt: string;
-    skill: string;
-    contractVersion: 1;
-    runId: string;
-    leaseGeneration: number;
-    attemptId: string;
-}>;
+}, z.core.$strip>;
 export type RunAdmission = z.infer<typeof runAdmissionSchema>;
 /** Lease: a worker has claimed the run. */
 export declare const runLeaseSchema: z.ZodObject<{
@@ -64,21 +48,7 @@ export declare const runLeaseSchema: z.ZodObject<{
     leaseGeneration: z.ZodNumber;
     workerId: z.ZodString;
     status: z.ZodLiteral<"leased">;
-}, "strip", z.ZodTypeAny, {
-    status: "leased";
-    contractVersion: 1;
-    runId: string;
-    leaseGeneration: number;
-    attemptId: string;
-    workerId: string;
-}, {
-    status: "leased";
-    contractVersion: 1;
-    runId: string;
-    leaseGeneration: number;
-    attemptId: string;
-    workerId: string;
-}>;
+}, z.core.$strip>;
 export type RunLease = z.infer<typeof runLeaseSchema>;
 /** Terminal: the run reached a final state. */
 export declare const runTerminalSchema: z.ZodObject<{
@@ -87,28 +57,17 @@ export declare const runTerminalSchema: z.ZodObject<{
     attemptId: z.ZodString;
     leaseGeneration: z.ZodNumber;
     skill: z.ZodString;
-    status: z.ZodEnum<["succeeded", "failed", "cancelled", "expired"]>;
+    status: z.ZodEnum<{
+        cancelled: "cancelled";
+        failed: "failed";
+        succeeded: "succeeded";
+        expired: "expired";
+    }>;
     completedAt: z.ZodString;
-}, "strip", z.ZodTypeAny, {
-    status: "cancelled" | "failed" | "succeeded" | "expired";
-    skill: string;
-    completedAt: string;
-    contractVersion: 1;
-    runId: string;
-    leaseGeneration: number;
-    attemptId: string;
-}, {
-    status: "cancelled" | "failed" | "succeeded" | "expired";
-    skill: string;
-    completedAt: string;
-    contractVersion: 1;
-    runId: string;
-    leaseGeneration: number;
-    attemptId: string;
-}>;
+}, z.core.$strip>;
 export type RunTerminal = z.infer<typeof runTerminalSchema>;
 /** Any single protocol message, discriminated by `status`. */
-export declare const runProtocolSchema: z.ZodDiscriminatedUnion<"status", [z.ZodObject<{
+export declare const runProtocolSchema: z.ZodDiscriminatedUnion<[z.ZodObject<{
     contractVersion: z.ZodLiteral<1>;
     runId: z.ZodString;
     attemptId: z.ZodString;
@@ -116,68 +75,27 @@ export declare const runProtocolSchema: z.ZodDiscriminatedUnion<"status", [z.Zod
     skill: z.ZodString;
     status: z.ZodLiteral<"admitted">;
     createdAt: z.ZodString;
-}, "strip", z.ZodTypeAny, {
-    status: "admitted";
-    createdAt: string;
-    skill: string;
-    contractVersion: 1;
-    runId: string;
-    leaseGeneration: number;
-    attemptId: string;
-}, {
-    status: "admitted";
-    createdAt: string;
-    skill: string;
-    contractVersion: 1;
-    runId: string;
-    leaseGeneration: number;
-    attemptId: string;
-}>, z.ZodObject<{
+}, z.core.$strip>, z.ZodObject<{
     contractVersion: z.ZodLiteral<1>;
     runId: z.ZodString;
     attemptId: z.ZodString;
     leaseGeneration: z.ZodNumber;
     workerId: z.ZodString;
     status: z.ZodLiteral<"leased">;
-}, "strip", z.ZodTypeAny, {
-    status: "leased";
-    contractVersion: 1;
-    runId: string;
-    leaseGeneration: number;
-    attemptId: string;
-    workerId: string;
-}, {
-    status: "leased";
-    contractVersion: 1;
-    runId: string;
-    leaseGeneration: number;
-    attemptId: string;
-    workerId: string;
-}>, z.ZodObject<{
+}, z.core.$strip>, z.ZodObject<{
     contractVersion: z.ZodLiteral<1>;
     runId: z.ZodString;
     attemptId: z.ZodString;
     leaseGeneration: z.ZodNumber;
     skill: z.ZodString;
-    status: z.ZodEnum<["succeeded", "failed", "cancelled", "expired"]>;
+    status: z.ZodEnum<{
+        cancelled: "cancelled";
+        failed: "failed";
+        succeeded: "succeeded";
+        expired: "expired";
+    }>;
     completedAt: z.ZodString;
-}, "strip", z.ZodTypeAny, {
-    status: "cancelled" | "failed" | "succeeded" | "expired";
-    skill: string;
-    completedAt: string;
-    contractVersion: 1;
-    runId: string;
-    leaseGeneration: number;
-    attemptId: string;
-}, {
-    status: "cancelled" | "failed" | "succeeded" | "expired";
-    skill: string;
-    completedAt: string;
-    contractVersion: 1;
-    runId: string;
-    leaseGeneration: number;
-    attemptId: string;
-}>]>;
+}, z.core.$strip>], "status">;
 export type RunProtocolMessage = z.infer<typeof runProtocolSchema>;
 /** Map the store's status vocabulary onto the protocol lifecycle. */
 export declare function protocolStateOf(status: ServerRunStatus): RunProtocolState;

package/dist/server/artifact-storage.d.ts

@@ -113,6 +113,17 @@ export declare class ArtifactStorage {
      *   <prefix>/skills/<org>/<slug>/<version>/manifest.json
      * The content-addressed object under bundles/ stays the read path (dedupe); these keys
      * are the durable, browsable history and are never deleted by orphan collection.
+     *
+     * RETENTION (hasna/apps#1671): these objects accumulate without bound by design — every
+     * version keeps a full copy — so the bucket lifecycle is the retention story, not the
+     * store. The intended rule (operated in the bucket's lifecycle config, outside this
+     * repository): expire `<prefix>/skills/` objects older than N days per the slot policy,
+     * while `<prefix>/bundles/` (content-addressed, deduped across versions and slugs) is
+     * NEVER expired by lifecycle — only the reference-guarded delete path removes those,
+     * because one object can serve many versions. A busy org that pushes thousands of
+     * versions should pair the lifecycle rule with a documented cap on retained versions
+     * per slug (e.g. the newest K, pruned by a maintenance job that deletes the version row
+     * and its two objects together); row deletion alone would strand the objects.
      * Returns the placement recorded on the version row; in db mode nothing is written.
      */
     putVersionObjects(orgId: string, slug: string, version: string, bytes: OwnedBytes, manifest: Record<string, unknown>, contentType?: string): Promise<{

package/dist/server/skills-api.d.ts

@@ -22,7 +22,15 @@ export declare class SkillRequestError extends Error {
 }
 /** SkillMeta shape for a published row, so a client can treat both kinds alike. */
 export declare function publishedSkillMeta(record: ServerSkillRecord): SkillMeta;
-export declare function publishedPayload(record: ServerSkillRecord): Record<string, unknown>;
+/**
+ * SkillMeta shape for a published row, plus a payload-level "alreadyPublished" flag:
+ * true when the accepted publish found the version already recorded with the same
+ * digest (an idempotent re-push), so the CLI can distinguish "published as X" from
+ * "already published as X" (hasna/apps#1671).
+ */
+export declare function publishedPayload(record: ServerSkillRecord, opts?: {
+    alreadyPublished?: boolean;
+}): Record<string, unknown>;
 /**
  * The HTTP ETag for a published row: the revision id, quoted exactly as RFC 9110 wants.
  * The quotes are load-bearing — a client that echoes the whole header value back as
@@ -143,7 +151,10 @@ interface ParsedPublish {
  * measured.
  */
 export declare function parsePublishRequest(request: Request, config: SkillsServerConfig): Promise<ParsedPublish>;
-export declare function storePublishedSkill(store: SkillsProductStore, artifactStorage: ArtifactStorage, principal: ApiPrincipal, parsed: ParsedPublish, expectedRevisionId?: string): Promise<ServerSkillRecord>;
+export declare function storePublishedSkill(store: SkillsProductStore, artifactStorage: ArtifactStorage, principal: ApiPrincipal, parsed: ParsedPublish, expectedRevisionId?: string): Promise<{
+    record: ServerSkillRecord;
+    alreadyPublished: boolean;
+}>;
 /**
  * Tombstone a published skill (todos d061fcda): the row survives with a tombstone
  * marker for `tombstoneWindowMs`, so reads answer 410 and a pulling client can

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hasna/skills",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Skills library for AI coding agents",
   "type": "module",
   "bin": {