fix(sdk-pypi): allow dry-run without token

packages/targets/sdk-pypi/src/index.test.ts

@@ -0,0 +1,34 @@
+import { fakeShipContext } from '@profullstack/sh1pt-core/testing';
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+
+const { execMock } = vi.hoisted(() => ({
+  execMock: vi.fn(),
+}));
+
+vi.mock('@profullstack/sh1pt-core', async () => ({
+  ...await vi.importActual<typeof import('@profullstack/sh1pt-core')>('@profullstack/sh1pt-core'),
+  exec: execMock,
+}));
+
+import adapter from './index.js';
+
+beforeEach(() => {
+  vi.clearAllMocks();
+});
+
+describe('sdk-pypi target adapter', () => {
+  it('does not require a token or invoke twine for dry-run shipping', async () => {
+    const secret = vi.fn(() => undefined);
+
+    await expect(adapter.ship(fakeShipContext({
+      dryRun: true,
+      secret,
+    }) as any, {})).resolves.toEqual({
+      id: 'dry-run',
+      meta: { repository: 'https://upload.pypi.org/legacy/' },
+    });
+
+    expect(secret).not.toHaveBeenCalled();
+    expect(execMock).not.toHaveBeenCalled();
+  });
+});

packages/targets/sdk-pypi/src/index.ts

@@ -34,22 +34,22 @@ export default defineTarget<Config>({
   },
 
   async ship(ctx, config) {
+    const repository = config.repository ?? 'https://upload.pypi.org/legacy/';
+    if (ctx.dryRun) {
+      ctx.log('pypi: dry-run — would upload distribution');
+      return { id: 'dry-run', meta: { repository } };
+    }
+
     const token = ctx.secret('PYPI_TOKEN');
     if (!token) {
       throw new Error('PYPI_TOKEN not set. Run: sh1pt secret set PYPI_TOKEN <token>');
     }
 
-    const repository = config.repository ?? 'https://upload.pypi.org/legacy/';
     const repoFlag = config.repository === 'testpypi'
       ? ['--repository', 'testpypi']
       : ['--repository-url', repository];
 
     ctx.log(`pypi: uploading to ${repository}`);
-    if (ctx.dryRun) {
-      ctx.log('pypi: dry-run — would upload distribution');
-      return { id: 'dry-run', meta: { repository } };
-    }
-
     const { stdout } = await exec(
       'twine',
       [