@synergenius/flow-weaver 0.17.3 → 0.17.5

package/dist/chevrotain-parser/trigger-cancel-parser.js

@@ -148,14 +148,17 @@ export function parseTriggerLine(input, warnings) {
     parserInstance.input = lexResult.tokens;
     const cst = parserInstance.triggerLine();
     if (parserInstance.errors.length > 0) {
-        const firstError = parserInstance.errors[0];
-        const truncatedInput = input.length > 60 ? input.substring(0, 60) + '...' : input;
-        warnings.push(`Failed to parse trigger line: "${truncatedInput}"\n` +
-            `  Error: ${firstError.message}\n` +
-            `  Expected format: @trigger event="name" or @trigger cron="expr"`);
+        // Don't warn here — return null so domain-specific handlers (e.g. CI/CD)
+        // get a chance to parse the trigger. The caller can warn if nothing handles it.
         return null;
     }
     const result = visitorInstance.visit(cst);
+    // Empty result means the parser consumed @trigger but found no event=/cron= assignments.
+    // Return null so the caller can delegate to domain-specific handlers (e.g. CI/CD triggers
+    // like @trigger push, @trigger pull_request, etc.)
+    if (!result.event && !result.cron) {
+        return null;
+    }
     // Validate cron expression
     if (result.cron && !CRON_REGEX.test(result.cron)) {
         warnings.push(`Invalid cron expression: "${result.cron}". Expected 5 fields (minute hour day month weekday).`);

package/dist/cli/commands/export.js

@@ -103,7 +103,9 @@ export async function exportCommand(input, options) {
         logger.section('Handler Preview');
         const handlerFile = result.files.find((f) => f.path.endsWith('handler.ts') ||
             f.path.endsWith(`${result.workflow}.ts`) ||
-            f.path.endsWith('index.ts'));
+            f.path.endsWith('index.ts') ||
+            f.path.endsWith('.yml') ||
+            f.path.endsWith('.yaml'));
         if (handlerFile) {
             // Show first 40 lines of handler
             const lines = handlerFile.content.split('\n');
@@ -114,6 +116,14 @@ export async function exportCommand(input, options) {
             }
         }
     }
+    // Show warnings about unsupported annotations
+    if (result.warnings && result.warnings.length > 0) {
+        logger.newline();
+        logger.section('Warnings');
+        for (const warning of result.warnings) {
+            logger.warn(warning);
+        }
+    }
     // Get deploy instructions from the target
     logger.newline();
     logger.section('Next Steps');

package/dist/cli/flow-weaver.mjs

@@ -17051,16 +17051,12 @@ function parseTriggerLine(input, warnings) {
   parserInstance8.input = lexResult.tokens;
   const cst = parserInstance8.triggerLine();
   if (parserInstance8.errors.length > 0) {
-    const firstError = parserInstance8.errors[0];
-    const truncatedInput = input.length > 60 ? input.substring(0, 60) + "..." : input;
-    warnings.push(
-      `Failed to parse trigger line: "${truncatedInput}"
-  Error: ${firstError.message}
-  Expected format: @trigger event="name" or @trigger cron="expr"`
-    );
     return null;
   }
   const result = visitorInstance8.visit(cst);
+  if (!result.event && !result.cron) {
+    return null;
+  }
   if (result.cron && !CRON_REGEX.test(result.cron)) {
     warnings.push(`Invalid cron expression: "${result.cron}". Expected 5 fields (minute hour day month weekday).`);
   }
@@ -25611,7 +25607,7 @@ var VERSION2;
 var init_generated_version = __esm({
   "src/generated-version.ts"() {
     "use strict";
-    VERSION2 = "0.17.3";
+    VERSION2 = "0.17.5";
   }
 });
 
@@ -35963,6 +35959,12 @@ var init_jsdoc_parser = __esm({
         const comment = (tag.getCommentText() || "").trim();
         const result = parseTriggerLine(`@trigger ${comment}`, warnings);
         if (result) {
+          const cicdKeywords = ["push", "pull_request", "dispatch", "tag", "schedule"];
+          if (result.event && cicdKeywords.includes(result.event)) {
+            warnings.push(
+              `@trigger event="${result.event}" is treated as an Inngest event trigger, not a CI/CD trigger. For CI/CD, use: @trigger ${result.event}`
+            );
+          }
           config2.trigger = config2.trigger || {};
           if (result.event) config2.trigger.event = result.event;
           if (result.cron) config2.trigger.cron = result.cron;
@@ -36677,7 +36679,8 @@ var init_parser2 = __esm({
         nodeTypes.push(...inferredNodeTypes);
         const workflows = this.extractWorkflows(sourceFile, nodeTypes, filePath, errors2, warnings);
         const patterns = this.extractPatterns(sourceFile, nodeTypes, filePath, errors2, warnings);
-        const result = { workflows, nodeTypes, patterns, errors: errors2, warnings };
+        const dedupedWarnings = [...new Set(warnings)];
+        const result = { workflows, nodeTypes, patterns, errors: errors2, warnings: dedupedWarnings };
         this.project.removeSourceFile(sourceFile);
         if (!externalNodeTypes?.length) {
           this.parseCache.set(filePath, {
@@ -36717,12 +36720,13 @@ var init_parser2 = __esm({
         const workflows = this.extractWorkflows(sourceFile, nodeTypes, virtualPath, errors2, warnings);
         const patterns = this.extractPatterns(sourceFile, nodeTypes, virtualPath, errors2, warnings);
         this.project.removeSourceFile(sourceFile);
+        const dedupedWarnings = [...new Set(warnings)];
         return {
           workflows,
           nodeTypes,
           patterns,
           errors: errors2,
-          warnings
+          warnings: dedupedWarnings
         };
       }
       clearCache() {
@@ -71463,7 +71467,7 @@ var init_base_target = __esm({
       },
       "slack-notify": {
         githubAction: "slackapi/slack-github-action@v1",
-        gitlabScript: ['curl -X POST -H "Content-type: application/json" --data "{\\"text\\":\\"Pipeline complete\\"}" $SLACK_WEBHOOK_URL'],
+        gitlabScript: [`curl -X POST -H 'Content-type: application/json' --data '{"text":"Pipeline complete"}' $SLACK_WEBHOOK_URL`],
         label: "Send Slack notification"
       },
       "health-check": {
@@ -71575,11 +71579,10 @@ var init_base_target = __esm({
         const stages = ast.options?.cicd?.stages;
         if (stages && stages.length > 0) {
           const depthMap = this.computeJobDepths(jobs);
-          for (const jc of jobConfigs || []) {
-            const job = jobs.find((j) => j.id === jc.id);
-            if (job && !job.stage) {
+          for (const job of jobs) {
+            if (!job.stage) {
               for (const s of stages) {
-                if (jc.id === s.name || jc.id.startsWith(s.name + "-") || jc.id.startsWith(s.name + "_")) {
+                if (job.id === s.name || job.id.startsWith(s.name + "-") || job.id.startsWith(s.name + "_")) {
                   job.stage = s.name;
                   break;
                 }
@@ -73354,6 +73357,18 @@ function parseJob(text, d, warnings) {
         jc.rules = jc.rules || [];
         jc.rules.push({ if: value2 });
         break;
+      case "when": {
+        jc.rules = jc.rules || [];
+        if (jc.rules.length === 0) jc.rules.push({});
+        jc.rules[jc.rules.length - 1].when = value2;
+        break;
+      }
+      case "changes": {
+        jc.rules = jc.rules || [];
+        if (jc.rules.length === 0) jc.rules.push({});
+        jc.rules[jc.rules.length - 1].changes = value2.split(",").map((s) => s.trim()).filter(Boolean);
+        break;
+      }
       case "reports": {
         jc.reports = jc.reports || [];
         for (const pair of value2.split(",")) {
@@ -73794,6 +73809,18 @@ function deploySsh(sshKey: string = ''): { result: string } { return { result: '
  */
 function deployS3(accessKey: string = '', secretKey: string = ''): { result: string } { return { result: 'deployed' }; }
 ` : "";
+    const stageAnnotations = hasDeploy ? ` * @stage test
+ * @stage build
+ * @stage deploy
+` : ` * @stage test
+ * @stage build
+`;
+    const jobAnnotations = hasDeploy ? ` * @job test retry=1
+ * @job build timeout="10m"
+ * @job deploy allow_failure=false
+` : ` * @job test retry=1
+ * @job build timeout="10m"
+`;
     return `/** @flowWeaver nodeType
  * @expression
  * @label Checkout code
@@ -73832,6 +73859,7 @@ ${deployStub}
  * @secret NPM_TOKEN - npm auth token${deploySecret}
  * @cache npm key="package-lock.json"
  *
+${stageAnnotations}${jobAnnotations} *
  * @node co checkout [job: "test"] [position: 270 0]
  * @node setup setupNode [job: "test"] [position: 540 0]
  * @node install npmInstall [job: "test"] [position: 810 0]
@@ -73996,6 +74024,15 @@ var cicdMultiEnvTemplate = {
     const name = opts.workflowName || "multiEnvPipeline";
     const envs = (opts.config?.environments || "staging,production").split(",").map((e) => e.trim());
     const envAnnotations = envs.map((env) => ` * @environment ${env} url="https://${env}.example.com"`).join("\n");
+    const stageAnnotations = ` * @stage test
+ * @stage build
+ * @stage deploy
+`;
+    const jobAnnotations = [
+      ` * @job test retry=1`,
+      ` * @job build timeout="10m"`,
+      ...envs.map((env) => ` * @job deploy-${env} allow_failure=${env === "staging" ? "true" : "false"}`)
+    ].join("\n");
     let x = 270;
     const nodeAnnotations = [];
     const pathParts = ["Start"];
@@ -74048,6 +74085,8 @@ ${envAnnotations}
  * @cache npm key="package-lock.json"
  * @artifact dist path="dist/" retention=3
  *
+${stageAnnotations}${jobAnnotations}
+ *
 ${nodeAnnotations.join("\n")}
  *
  * @path ${pathParts.join(" -> ")}
@@ -107919,7 +107958,8 @@ async function exportSingleWorkflowViaRegistry(target, inputPath, outputDir, isD
     target: options.target,
     files,
     workflow: workflow.name,
-    description: workflow.description
+    description: workflow.description,
+    warnings: artifacts.warnings
   };
 }
 async function exportMultiWorkflowViaRegistry(target, inputPath, outputDir, isDryRun, options) {
@@ -107981,7 +108021,8 @@ async function exportMultiWorkflowViaRegistry(target, inputPath, outputDir, isDr
     target: options.target,
     files,
     workflow: serviceName,
-    workflows: selectedWorkflows.map((w) => w.name)
+    workflows: selectedWorkflows.map((w) => w.name),
+    warnings: artifacts.warnings
   };
 }
 async function compileToOutput(inputPath, functionName, outputDir, production) {
@@ -108070,7 +108111,7 @@ async function exportCommand(input, options) {
     logger.newline();
     logger.section("Handler Preview");
     const handlerFile = result.files.find(
-      (f) => f.path.endsWith("handler.ts") || f.path.endsWith(`${result.workflow}.ts`) || f.path.endsWith("index.ts")
+      (f) => f.path.endsWith("handler.ts") || f.path.endsWith(`${result.workflow}.ts`) || f.path.endsWith("index.ts") || f.path.endsWith(".yml") || f.path.endsWith(".yaml")
     );
     if (handlerFile) {
       const lines = handlerFile.content.split("\n");
@@ -108081,6 +108122,13 @@ async function exportCommand(input, options) {
       }
     }
   }
+  if (result.warnings && result.warnings.length > 0) {
+    logger.newline();
+    logger.section("Warnings");
+    for (const warning of result.warnings) {
+      logger.warn(warning);
+    }
+  }
   logger.newline();
   logger.section("Next Steps");
   const { createTargetRegistry: createTargetRegistry2 } = await Promise.resolve().then(() => (init_deployment(), deployment_exports));
@@ -109257,7 +109305,7 @@ function displayInstalledPackage(pkg) {
 
 // src/cli/index.ts
 init_error_utils();
-var version2 = true ? "0.17.3" : "0.0.0-dev";
+var version2 = true ? "0.17.5" : "0.0.0-dev";
 var program2 = new Command();
 program2.name("flow-weaver").description("Flow Weaver Annotations - Compile and validate workflow files").option("-v, --version", "Output the current version").option("--no-color", "Disable colors").option("--color", "Force colors").on("option:version", () => {
   logger.banner(version2);

package/dist/deployment/targets/base.d.ts

@@ -67,6 +67,8 @@ export interface ExportArtifacts {
     workflowName: string;
     /** Entry point file */
     entryPoint: string;
+    /** Warnings about unsupported or dropped annotations */
+    warnings?: string[];
 }
 /**
  * Deployment instructions

package/dist/export/index.d.ts

@@ -55,6 +55,8 @@ export interface ExportResult {
     workflows?: string[];
     /** OpenAPI spec (if generated) */
     openApiSpec?: object;
+    /** Warnings about unsupported or dropped annotations */
+    warnings?: string[];
 }
 /**
  * Export a workflow for deployment.

package/dist/export/index.js

@@ -112,6 +112,7 @@ async function exportSingleWorkflowViaRegistry(target, inputPath, outputDir, isD
         files,
         workflow: workflow.name,
         description: workflow.description,
+        warnings: artifacts.warnings,
     };
 }
 /**
@@ -177,6 +178,7 @@ async function exportMultiWorkflowViaRegistry(target, inputPath, outputDir, isDr
         files,
         workflow: serviceName,
         workflows: selectedWorkflows.map((w) => w.name),
+        warnings: artifacts.warnings,
     };
 }
 /**

package/dist/extensions/cicd/base-target.js

@@ -79,7 +79,7 @@ export const NODE_ACTION_MAP = {
     },
     'slack-notify': {
         githubAction: 'slackapi/slack-github-action@v1',
-        gitlabScript: ['curl -X POST -H "Content-type: application/json" --data "{\\\"text\\\":\\\"Pipeline complete\\\"}" $SLACK_WEBHOOK_URL'],
+        gitlabScript: ["curl -X POST -H 'Content-type: application/json' --data '{\"text\":\"Pipeline complete\"}' $SLACK_WEBHOOK_URL"],
         label: 'Send Slack notification',
     },
     'health-check': {
@@ -224,11 +224,11 @@ export class BaseCICDTarget extends BaseExportTarget {
         const stages = ast.options?.cicd?.stages;
         if (stages && stages.length > 0) {
             const depthMap = this.computeJobDepths(jobs);
-            for (const jc of jobConfigs || []) {
-                const job = jobs.find(j => j.id === jc.id);
-                if (job && !job.stage) {
+            // Match ALL jobs by name prefix to stages, not just configured ones
+            for (const job of jobs) {
+                if (!job.stage) {
                     for (const s of stages) {
-                        if (jc.id === s.name || jc.id.startsWith(s.name + '-') || jc.id.startsWith(s.name + '_')) {
+                        if (job.id === s.name || job.id.startsWith(s.name + '-') || job.id.startsWith(s.name + '_')) {
                             job.stage = s.name;
                             break;
                         }

package/dist/extensions/cicd/tag-handler.js

@@ -326,6 +326,22 @@ function parseJob(text, d, warnings) {
                 jc.rules = jc.rules || [];
                 jc.rules.push({ if: value });
                 break;
+            case 'when': {
+                // Modifier: sets `when` on the last rule, or creates a standalone rule
+                jc.rules = jc.rules || [];
+                if (jc.rules.length === 0)
+                    jc.rules.push({});
+                jc.rules[jc.rules.length - 1].when = value;
+                break;
+            }
+            case 'changes': {
+                // Modifier: sets `changes` on the last rule
+                jc.rules = jc.rules || [];
+                if (jc.rules.length === 0)
+                    jc.rules.push({});
+                jc.rules[jc.rules.length - 1].changes = value.split(',').map(s => s.trim()).filter(Boolean);
+                break;
+            }
             case 'reports': {
                 jc.reports = jc.reports || [];
                 for (const pair of value.split(',')) {

package/dist/extensions/cicd/templates/cicd-multi-env.js

@@ -35,6 +35,12 @@ export const cicdMultiEnvTemplate = {
         const envAnnotations = envs
             .map((env) => ` * @environment ${env} url="https://${env}.example.com"`)
             .join('\n');
+        const stageAnnotations = ` * @stage test\n * @stage build\n * @stage deploy\n`;
+        const jobAnnotations = [
+            ` * @job test retry=1`,
+            ` * @job build timeout="10m"`,
+            ...envs.map((env) => ` * @job deploy-${env} allow_failure=${env === 'staging' ? 'true' : 'false'}`),
+        ].join('\n');
         let x = 270;
         const nodeAnnotations = [];
         const pathParts = ['Start'];
@@ -89,6 +95,8 @@ ${envAnnotations}
  * @cache npm key="package-lock.json"
  * @artifact dist path="dist/" retention=3
  *
+${stageAnnotations}${jobAnnotations}
+ *
 ${nodeAnnotations.join('\n')}
  *
  * @path ${pathParts.join(' -> ')}

package/dist/extensions/cicd/templates/cicd-test-deploy.js

@@ -78,6 +78,12 @@ function deploySsh(sshKey: string = ''): { result: string } { return { result: '
 function deployS3(accessKey: string = '', secretKey: string = ''): { result: string } { return { result: 'deployed' }; }
 `
             : '';
+        const stageAnnotations = hasDeploy
+            ? ` * @stage test\n * @stage build\n * @stage deploy\n`
+            : ` * @stage test\n * @stage build\n`;
+        const jobAnnotations = hasDeploy
+            ? ` * @job test retry=1\n * @job build timeout="10m"\n * @job deploy allow_failure=false\n`
+            : ` * @job test retry=1\n * @job build timeout="10m"\n`;
         return `/** @flowWeaver nodeType
  * @expression
  * @label Checkout code
@@ -116,6 +122,7 @@ ${deployStub}
  * @secret NPM_TOKEN - npm auth token${deploySecret}
  * @cache npm key="package-lock.json"
  *
+${stageAnnotations}${jobAnnotations} *
  * @node co checkout [job: "test"] [position: 270 0]
  * @node setup setupNode [job: "test"] [position: 540 0]
  * @node install npmInstall [job: "test"] [position: 810 0]

package/dist/generated-version.d.ts

@@ -1,2 +1,2 @@
-export declare const VERSION = "0.17.3";
+export declare const VERSION = "0.17.5";
 //# sourceMappingURL=generated-version.d.ts.map

package/dist/generated-version.js

@@ -1,3 +1,3 @@
 // Auto-generated by scripts/generate-version.ts — do not edit manually
-export const VERSION = '0.17.3';
+export const VERSION = '0.17.5';
 //# sourceMappingURL=generated-version.js.map

package/dist/jsdoc-parser.js

@@ -1055,6 +1055,12 @@ export class JSDocParser {
         // Try core FW trigger parsing first (event= and/or cron=)
         const result = parseTriggerLine(`@trigger ${comment}`, warnings);
         if (result) {
+            // Warn if the event name matches a CI/CD trigger keyword (likely user error)
+            const cicdKeywords = ['push', 'pull_request', 'dispatch', 'tag', 'schedule'];
+            if (result.event && cicdKeywords.includes(result.event)) {
+                warnings.push(`@trigger event="${result.event}" is treated as an Inngest event trigger, not a CI/CD trigger. ` +
+                    `For CI/CD, use: @trigger ${result.event}`);
+            }
             // Merge: multiple @trigger tags accumulate (event + cron can be separate tags)
             config.trigger = config.trigger || {};
             if (result.event)

package/dist/parser.js

@@ -187,7 +187,9 @@ export class AnnotationParser {
         nodeTypes.push(...inferredNodeTypes);
         const workflows = this.extractWorkflows(sourceFile, nodeTypes, filePath, errors, warnings);
         const patterns = this.extractPatterns(sourceFile, nodeTypes, filePath, errors, warnings);
-        const result = { workflows, nodeTypes, patterns, errors, warnings };
+        // Deduplicate warnings (extractWorkflowSignatures + extractWorkflows both parse JSDoc)
+        const dedupedWarnings = [...new Set(warnings)];
+        const result = { workflows, nodeTypes, patterns, errors, warnings: dedupedWarnings };
         // Clean up source file to prevent ts-morph Project bloat
         // (results are captured in the returned AST, source file is no longer needed)
         this.project.removeSourceFile(sourceFile);
@@ -238,12 +240,14 @@ export class AnnotationParser {
         // Clean up virtual source file to prevent memory bloat
         // (tests create many unique virtual paths that accumulate)
         this.project.removeSourceFile(sourceFile);
+        // Deduplicate warnings (extractWorkflowSignatures + extractWorkflows both parse JSDoc)
+        const dedupedWarnings = [...new Set(warnings)];
         return {
             workflows,
             nodeTypes,
             patterns,
             errors,
-            warnings,
+            warnings: dedupedWarnings,
         };
     }
     clearCache() {

package/docs/reference/jsdoc-grammar.md

@@ -572,7 +572,9 @@ Configures per-job settings. The name must match a `[job: "name"]` attribute use
 jobTag         ::= "@job" IDENTIFIER { IDENTIFIER "=" ( STRING | IDENTIFIER | INTEGER ) }
 ```
 
-Recognized keys: `retry` (number), `allow_failure` (boolean), `timeout` (string), `runner` (string), `tags` (comma-list), `coverage` (string), `reports` (comma-list of type=path), `rules` (string), `extends` (string), `before_script` (comma-list), `variables` (comma-list of KEY=VALUE).
+Recognized keys: `retry` (number), `allow_failure` (boolean), `timeout` (string), `runner` (string), `tags` (comma-list), `coverage` (string), `reports` (comma-list of type=path), `rules` (string), `when` (rule modifier), `changes` (rule modifier, comma-list), `extends` (string), `before_script` (comma-list), `variables` (comma-list of KEY=VALUE).
+
+The `when` and `changes` keys are rule modifiers: they apply to the most recently declared `rules` entry for that job. If no `rules` entry exists yet, one is created automatically.
 
 **Examples:**
 
@@ -580,6 +582,8 @@ Recognized keys: `retry` (number), `allow_failure` (boolean), `timeout` (string)
 @job build retry=2 timeout="10m"
 @job test-unit coverage='/Coverage: (\d+)%/' reports="junit=test-results.xml"
 @job deploy allow_failure=true rules="$CI_COMMIT_BRANCH == main"
+@job deploy rules="$CI_COMMIT_BRANCH == main" when=manual
+@job deploy rules="$CI_COMMIT_TAG" changes="src/**,lib/**"
 @job lint tags="docker,linux" extends=".base-lint"
 ```
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@synergenius/flow-weaver",
-  "version": "0.17.3",
+  "version": "0.17.5",
   "description": "Deterministic workflow compiler for AI agents. Compiles to standalone TypeScript, no runtime dependencies.",
   "private": false,
   "type": "module",