Fix wasm/native compiler divergence, add smoke workflow golden tests

cmd/gh-aw-wasm/main.go

@@ -3,6 +3,7 @@
 package main
 
 import (
+	"strings"
 	"syscall/js"
 
 	"github.com/github/gh-aw/pkg/parser"
@@ -87,9 +88,13 @@ func doCompile(markdown string, files map[string][]byte, filename string) (js.Va
 		defer parser.ClearVirtualFiles()
 	}
 
+	// Derive workflow identifier from filename for fuzzy cron schedule scattering
+	identifier := strings.TrimSuffix(filename, ".md")
+
 	compiler := workflow.NewCompiler(
 		workflow.WithNoEmit(true),
 		workflow.WithSkipValidation(true),
+		workflow.WithWorkflowIdentifier(identifier),
 	)
 
 	// Parse directly from string — no temp files needed

pkg/parser/virtual_fs.go

@@ -6,3 +6,10 @@ import "os"
 // In wasm builds, this is overridden to read from a virtual filesystem
 // populated by the browser via SetVirtualFiles.
 var readFileFunc = os.ReadFile
+
+// ReadFile reads a file using the parser's file reading function, which
+// checks the virtual filesystem first in wasm builds. Use this instead of
+// os.ReadFile when reading files that may be provided as virtual files.
+func ReadFile(path string) ([]byte, error) {
+	return readFileFunc(path)
+}

pkg/workflow/compiler_jobs.go

@@ -125,10 +125,18 @@ func (c *Compiler) getReferencedCustomJobs(content string, customJobs map[string
 func (c *Compiler) buildJobs(data *WorkflowData, markdownPath string) error {
 	compilerJobsLog.Printf("Building jobs for workflow: %s", markdownPath)
 
-	// Try to read frontmatter to determine event types for safe events check
+	// Try to read frontmatter to determine event types for safe events check.
+	// Use contentOverride first (set by ParseWorkflowString for wasm/string API mode),
+	// then fall back to reading from disk.
 	var frontmatter map[string]any
-	if content, err := os.ReadFile(markdownPath); err == nil {
-		if result, err := parser.ExtractFrontmatterFromContent(string(content)); err == nil {
+	var rawContent string
+	if c.contentOverride != "" {
+		rawContent = c.contentOverride
+	} else if diskContent, err := os.ReadFile(markdownPath); err == nil {
+		rawContent = string(diskContent)
+	}
+	if rawContent != "" {
+		if result, err := parser.ExtractFrontmatterFromContent(rawContent); err == nil {
 			frontmatter = result.Frontmatter
 		}
 	}

pkg/workflow/compiler_orchestrator_engine.go

@@ -122,7 +122,7 @@ func (c *Compiler) setupEngineAndImports(result *parser.FrontmatterResult, clean
 			fmt.Fprintf(os.Stderr, "WARNING: Skipping security scan for unresolvable import '%s': %v\n", importedFile, resolveErr)
 			continue
 		}
-		importContent, readErr := os.ReadFile(fullPath)
+		importContent, readErr := parser.ReadFile(fullPath)
 		if readErr != nil {
 			orchestratorEngineLog.Printf("Skipping security scan for unreadable import: %s: %v", fullPath, readErr)
 			fmt.Fprintf(os.Stderr, "WARNING: Skipping security scan for unreadable import '%s' (resolved path: %s): %v\n", importedFile, fullPath, readErr)

pkg/workflow/compiler_string_api.go

@@ -14,6 +14,8 @@ import (
 func (c *Compiler) CompileToYAML(workflowData *WorkflowData, markdownPath string) (string, error) {
 	c.markdownPath = markdownPath
 	c.skipHeader = true
+	// Clear contentOverride after compilation (set by ParseWorkflowString)
+	defer func() { c.contentOverride = "" }()
 
 	startTime := time.Now()
 	defer func() {
@@ -51,9 +53,9 @@ func (c *Compiler) ParseWorkflowString(content string, virtualPath string) (*Wor
 
 	cleanPath := filepath.Clean(virtualPath)
 
-	// Store content so downstream code can use it instead of reading from disk
+	// Store content so downstream code can use it instead of reading from disk.
+	// Cleared in CompileToYAML after compilation completes.
 	c.contentOverride = content
-	defer func() { c.contentOverride = "" }()
 
 	// Enable inline prompt mode for string-based compilation (Wasm/browser)
 	// since runtime-import macros cannot resolve without filesystem access