@michelangelo-ai/rpc 0.8.0 → 0.9.0

package/__tests__/handlers.test.ts

@@ -8,6 +8,7 @@ vi.mock('../services', () => ({ getServices: mockGetServices }));
 const createPipelineRun = vi.fn().mockResolvedValue({});
 const updatePipelineRun = vi.fn().mockResolvedValue({});
 const updateTriggerRun = vi.fn().mockResolvedValue({});
+const deletePipeline = vi.fn().mockResolvedValue({});
 
 mockGetServices.mockResolvedValue(
   new Proxy({} as Record<string, Record<string, unknown>>, {
@@ -18,6 +19,9 @@ mockGetServices.mockResolvedValue(
       if (serviceName === 'TriggerRunService') {
         return { updateTriggerRun };
       }
+      if (serviceName === 'PipelineService') {
+        return { deletePipeline };
+      }
       return new Proxy({}, { get: () => vi.fn().mockResolvedValue({}) });
     },
   })
@@ -64,4 +68,18 @@ describe('rpc handlers — mutation envelope wrapping', () => {
 
     expect((record.spec as Record<string, unknown>).actor).toBeUndefined();
   });
+
+  it('DeletePipeline extracts name/namespace from the full record', async () => {
+    const handlers = await getRpcHandlers();
+    const record = {
+      metadata: { name: 'eval-pipeline', namespace: 'ma-dev-test' },
+      spec: { owner: { name: 'me' } },
+    };
+    await handlers.DeletePipeline(record as never);
+
+    expect(deletePipeline).toHaveBeenCalledWith(
+      { name: 'eval-pipeline', namespace: 'ma-dev-test' },
+      undefined
+    );
+  });
 });

package/handlers.ts

@@ -15,6 +15,21 @@ function unary<Fn>(fn: Fn): ExtractUnaryRpc<Fn> {
   return fn as unknown as ExtractUnaryRpc<Fn>;
 }
 
+// Delete<CRD>Request messages are uniformly { name, namespace, deleteOptions? } across every
+// service, while actions send the full CRD record (metadata/spec/status/typeMeta). Wrapping a
+// generated deleteX method here lets every Delete* handler share this reshape instead of each
+// entity config repeating it as mutation middleware.
+function deleteCrd<Req extends { name: string; namespace: string }, Res>(
+  deleteFn: (request: Req, headers?: Record<string, string>) => Promise<Res>
+) {
+  return (
+    record: { metadata: { name: string; namespace: string } },
+    headers?: Record<string, string>
+  ) =>
+    // cast: Req may carry additional optional fields (e.g. deleteOptions) beyond name/namespace
+    deleteFn({ name: record.metadata.name, namespace: record.metadata.namespace } as Req, headers);
+}
+
 async function createHandlers() {
   const services = await getServices();
 
@@ -27,6 +42,7 @@ async function createHandlers() {
     GetProject: unary(services.ProjectService.getProject),
     GetPipeline: unary(services.PipelineService.getPipeline),
     ListPipeline: unary(services.PipelineService.listPipeline),
+    DeletePipeline: deleteCrd(services.PipelineService.deletePipeline),
     ListPipelineRun: unary(services.PipelineRunService.listPipelineRun),
     GetPipelineRun: unary(services.PipelineRunService.getPipelineRun),
     ListTriggerRun: unary(services.TriggerRunService.listTriggerRun),

package/package.json

@@ -2,7 +2,7 @@
   "name": "@michelangelo-ai/rpc",
   "license": "Apache-2.0",
   "type": "module",
-  "version": "0.8.0",
+  "version": "0.9.0",
   "repository": {
     "type": "git",
     "url": "https://github.com/michelangelo-ai/michelangelo"
@@ -17,7 +17,7 @@
   },
   "dependencies": {
     "@bufbuild/protobuf": "^2.2.5",
-    "@michelangelo-ai/core": "0.8.0"
+    "@michelangelo-ai/core": "0.9.0"
   },
   "exports": {
     ".": {