url handling

Author: alexanderkirtzel

.changeset/fix-cli-url-handling.md

@@ -0,0 +1,6 @@
+---
+'@walkeros/cli': patch
+---
+
+Fix URL handling in resolveAsset - URLs are now passed through unchanged instead
+of being mangled into invalid filesystem paths

.github/workflows/release.yml

@@ -94,7 +94,7 @@ jobs:
             exit $EXIT_CODE
           fi
           # Extract published packages for PR comment
-          PUBLISHED=$(echo "$OUTPUT" | grep -E "^@walkeros/|^walkeros" | head -20 || true)
+          PUBLISHED=$(echo "$OUTPUT" | grep -E "^@walkeros/|^walkeros" | head -20 | sed 's/^/- /' || true)
           echo "published<<EOF" >> $GITHUB_OUTPUT
           echo "$PUBLISHED" >> $GITHUB_OUTPUT
           echo "EOF" >> $GITHUB_OUTPUT
@@ -204,6 +204,7 @@ jobs:
 
             BODY="$EMOJI **$TITLE**
 
+          **Packages**
           ${{ steps.publish.outputs.published }}
 
           Install: \`npm i @walkeros/core$INSTALL_TAG\`$DOCKER_SECTION"

packages/cli/src/__tests__/core/asset-resolver.test.ts

@@ -0,0 +1,15 @@
+import { resolveAsset } from '../../core/asset-resolver.js';
+
+describe('resolveAsset', () => {
+  describe('URL handling', () => {
+    it('should preserve HTTPS URLs as-is', () => {
+      const url = 'https://www.walkeros.io/flows/gcp-bigquery.json';
+      expect(resolveAsset(url, 'config')).toBe(url);
+    });
+
+    it('should preserve HTTP URLs as-is', () => {
+      const url = 'http://example.com/config.json';
+      expect(resolveAsset(url, 'config')).toBe(url);
+    });
+  });
+});

packages/cli/src/core/asset-resolver.ts

@@ -8,6 +8,7 @@
 import { fileURLToPath } from 'url';
 import { existsSync } from 'fs';
 import path from 'path';
+import { isUrl } from '../config/utils.js';
 
 /**
  * Cached asset directory to avoid repeated filesystem checks
@@ -65,6 +66,11 @@ export function resolveAsset(
   assetType: AssetType,
   baseDir?: string,
 ): string {
+  // URL → pass through unchanged
+  if (isUrl(assetPath)) {
+    return assetPath;
+  }
+
   // Bare name → package asset (examples directory)
   if (!assetPath.includes('/') && !assetPath.includes('\\')) {
     const assetDir = getAssetDir();