veryfront 0.1.73 → 0.1.75

package/src/src/transforms/pipeline/stages/ssr-vf-modules/path-resolver.ts

@@ -20,6 +20,7 @@ import {
 export async function tryReadWithExtensions(
   fs: ReturnType<typeof createFileSystem>,
   basePath: string,
+  existsFn: (path: string) => Promise<boolean> = exists,
 ): Promise<{ sourcePath: string; content: string } | null> {
   // Try all extensions, including .src versions for embedded sources
   const allExtensions = [
@@ -30,7 +31,7 @@ export async function tryReadWithExtensions(
   for (const ext of allExtensions) {
     const sourcePath = basePath + ext;
     try {
-      if (await exists(sourcePath)) {
+      if (await existsFn(sourcePath)) {
         const content = await fs.readTextFile(sourcePath);
         return { sourcePath, content };
       }
@@ -47,6 +48,7 @@ export async function tryReadWithExtensions(
 export async function resolveFrameworkFile(
   vfModulePath: string,
   fs: ReturnType<typeof createFileSystem>,
+  existsFn: (path: string) => Promise<boolean> = exists,
 ): Promise<{ sourcePath: string; content: string } | null> {
   const pathWithoutPrefix = vfModulePath
     .replace(/^\/_vf_modules\//, "")
@@ -78,7 +80,7 @@ export async function resolveFrameworkFile(
       fullPath: pathWithPrefixDir,
     });
 
-    const withPrefix = await tryReadWithExtensions(fs, pathWithPrefixDir);
+    const withPrefix = await tryReadWithExtensions(fs, pathWithPrefixDir, existsFn);
     if (withPrefix) {
       logger.debug(`${LOG_PREFIX} Found with prefix`, { sourcePath: withPrefix.sourcePath });
       return withPrefix;
@@ -93,7 +95,7 @@ export async function resolveFrameworkFile(
       fullPath: pathWithoutPrefixDir,
     });
 
-    const withoutPrefix = await tryReadWithExtensions(fs, pathWithoutPrefixDir);
+    const withoutPrefix = await tryReadWithExtensions(fs, pathWithoutPrefixDir, existsFn);
     if (withoutPrefix) {
       logger.debug(`${LOG_PREFIX} Found without prefix`, { sourcePath: withoutPrefix.sourcePath });
       return withoutPrefix;
@@ -118,7 +120,10 @@ export async function resolveFrameworkFile(
  * then falls back to regular src/. This matches resolveFrameworkFile's behavior
  * and ensures consistent path resolution for cycle detection.
  */
-export async function resolveVeryfrontSourcePath(specifier: string): Promise<string | null> {
+export async function resolveVeryfrontSourcePath(
+  specifier: string,
+  existsFn: (path: string) => Promise<boolean> = exists,
+): Promise<string | null> {
   if (!specifier.startsWith("#veryfront/")) return null;
 
   const mappedTarget = resolveInternalModuleTarget(specifier);
@@ -143,13 +148,13 @@ export async function resolveVeryfrontSourcePath(specifier: string): Promise<str
       // Try exact path with .src suffix first (for embedded sources)
       try {
         const srcPath = basePath + ".src";
-        if (await exists(srcPath)) return srcPath;
+        if (await existsFn(srcPath)) return srcPath;
       } catch (_) {
         /* expected: file may not exist at this path */
       }
       // Try exact path
       try {
-        if (await exists(basePath)) return basePath;
+        if (await existsFn(basePath)) return basePath;
       } catch (_) {
         /* expected: file may not exist at this path */
       }
@@ -166,7 +171,7 @@ export async function resolveVeryfrontSourcePath(specifier: string): Promise<str
     for (const ext of allExtensions) {
       const pathWithExt = basePath + ext;
       try {
-        if (await exists(pathWithExt)) return pathWithExt;
+        if (await existsFn(pathWithExt)) return pathWithExt;
       } catch (_) {
         /* expected: file may not exist at this path */
       }
@@ -176,7 +181,7 @@ export async function resolveVeryfrontSourcePath(specifier: string): Promise<str
     for (const ext of allExtensions) {
       const indexPath = join(basePath, "index" + ext);
       try {
-        if (await exists(indexPath)) return indexPath;
+        if (await existsFn(indexPath)) return indexPath;
       } catch (_) {
         /* expected: file may not exist at this path */
       }
@@ -197,6 +202,7 @@ export async function resolveRelativeFrameworkImport(
   specifier: string,
   fromSourcePath: string,
   _fs: ReturnType<typeof createFileSystem>,
+  existsFn: (path: string) => Promise<boolean> = exists,
 ): Promise<string | null> {
   const fromDir = fromSourcePath.substring(0, fromSourcePath.lastIndexOf("/"));
   const parts = fromDir.split("/").filter(Boolean);
@@ -219,7 +225,7 @@ export async function resolveRelativeFrameworkImport(
   if (/\.(tsx?|jsx?|mjs)$/.test(specifier)) {
     // Try exact path first
     try {
-      if (await exists(basePath)) return basePath;
+      if (await existsFn(basePath)) return basePath;
     } catch (_) {
       /* expected: file may not exist at this path */
     }
@@ -227,7 +233,7 @@ export async function resolveRelativeFrameworkImport(
     // Try with .src suffix for embedded sources
     try {
       const srcPath = basePath + ".src";
-      if (await exists(srcPath)) return srcPath;
+      if (await existsFn(srcPath)) return srcPath;
     } catch (_) {
       /* expected: file may not exist at this path */
     }
@@ -245,7 +251,7 @@ export async function resolveRelativeFrameworkImport(
   for (const ext of allExtensions) {
     const pathWithExt = basePath + ext;
     try {
-      if (await exists(pathWithExt)) return pathWithExt;
+      if (await existsFn(pathWithExt)) return pathWithExt;
     } catch (_) {
       /* expected: file may not exist at this path */
     }
@@ -255,7 +261,7 @@ export async function resolveRelativeFrameworkImport(
   for (const ext of allExtensions) {
     const indexPath = join(basePath, "index" + ext);
     try {
-      if (await exists(indexPath)) return indexPath;
+      if (await existsFn(indexPath)) return indexPath;
     } catch (_) {
       /* expected: file may not exist at this path */
     }

package/src/src/utils/index.ts

@@ -3,7 +3,16 @@
  * (breakpoints, timeouts, HTTP codes), hashing, memoization, and feature flags.
  *
  * @module utils
+ *
+ * @example Structured logging
+ * ```ts
+ * import { serverLogger } from "veryfront/utils";
+ *
+ * serverLogger.info("Booting server", { project_id: "proj_123" });
+ * ```
  */
+import "../../_dnt.polyfills.js";
+
 
 export {
   type GlobalWithBun,
@@ -17,11 +26,13 @@ export {
 export {
   agentLogger,
   bundlerLogger,
+  createJobUserLogger,
   logger,
   refreshLoggerConfig,
   rendererLogger,
   serverLogger,
 } from "./logger/index.js";
+export type { Logger } from "./logger/index.js";
 
 export {
   BREAKPOINT_LG,

package/src/src/utils/logger/index.ts

@@ -12,6 +12,7 @@ export {
   agentLogger,
   bundlerLogger,
   cliLogger,
+  createJobUserLogger,
   createRequestLogger,
   getBaseLogger,
   getDefaultLevel,

package/src/src/utils/logger/logger.ts

@@ -60,6 +60,12 @@ export interface LogEntry {
   release_id?: string;
   branch_id?: string;
   branch_name?: string;
+  job_id?: string;
+  batch_id?: string;
+  job_target?: string;
+  task?: string;
+  event_kind?: string;
+  user_visible?: string;
   // Duration for timed operations
   /** @deprecated Use `duration_ms` instead. Kept for Grafana dashboard transition. Planned removal after Grafana dashboard migration is complete. */
   durationMs?: number;
@@ -314,6 +320,12 @@ class ConsoleLogger implements Logger {
     extractToEntryField(entry, mergedContext, "release_id", (v) => String(v));
     extractToEntryField(entry, mergedContext, "branch_id", (v) => String(v));
     extractToEntryField(entry, mergedContext, "branch_name", (v) => String(v));
+    extractToEntryField(entry, mergedContext, "job_id", (v) => String(v));
+    extractToEntryField(entry, mergedContext, "batch_id", (v) => String(v));
+    extractToEntryField(entry, mergedContext, "job_target", (v) => String(v));
+    extractToEntryField(entry, mergedContext, "task", (v) => String(v));
+    extractToEntryField(entry, mergedContext, "event_kind", (v) => String(v));
+    extractToEntryField(entry, mergedContext, "user_visible", (v) => String(v));
     extractToEntryField(entry, mergedContext, "duration_ms", (v) => Number(v));
 
     // Emit snake_case aliases for camelCase fields (transition period)
@@ -568,3 +580,25 @@ export function createRequestLogger(
 ): Logger {
   return baseLogger.child(requestContext);
 }
+
+export function createJobUserLogger(
+  baseLogger: Logger,
+  jobContext: {
+    projectId: string;
+    jobId: string;
+    task: string;
+    batchId?: string | null;
+    jobTarget?: string | null;
+    eventKind?: string;
+  },
+): Logger {
+  return baseLogger.child({
+    project_id: jobContext.projectId,
+    job_id: jobContext.jobId,
+    ...(jobContext.batchId ? { batch_id: jobContext.batchId } : {}),
+    ...(jobContext.jobTarget ? { job_target: jobContext.jobTarget } : {}),
+    task: jobContext.task,
+    event_kind: jobContext.eventKind ?? "job_user_log",
+    user_visible: "true",
+  });
+}