backend-manager 5.0.133 → 5.0.135

package/CLAUDE.md

@@ -467,16 +467,16 @@ npx bm test
 ```
 
 ### Log Files
-Both `npx bm emulator` and `npx bm test` automatically save all output to log files in the project directory while still streaming to the console:
-- **`emulator.log`** — Full emulator output (Firebase emulator + Cloud Functions logs)
-- **`test.log`** — Test runner output (when running against an existing emulator)
+BEM CLI commands automatically save all output to log files in `functions/` while still streaming to the console:
+- **`functions/serve.log`** — Output from `npx bm serve` (Firebase serve)
+- **`functions/emulator.log`** — Full emulator output (Firebase emulator + Cloud Functions logs)
+- **`functions/test.log`** — Test runner output (when running against an existing emulator)
+- **`functions/logs.log`** — Cloud Function logs from `npx bm logs:read` or `npx bm logs:tail` (raw JSON for `read`, streaming text for `tail`)
 
 When `npx bm test` starts its own emulator, logs go to `emulator.log` (since it delegates to the emulator command). When running against an already-running emulator, logs go to `test.log`.
 
 These files are overwritten on each run and are gitignored (`*.log`). Use them to search for errors, debug webhook pipelines, or review full function output after a test run.
 
-- **`logs.log`** — Cloud Function logs from `npx bm logs:read` or `npx bm logs:tail` (raw JSON for `read`, streaming text for `tail`)
-
 ### Filtering Tests
 ```bash
 npx bm test rules/             # Run rules tests (both BEM and project)
@@ -630,7 +630,7 @@ npx bm logs:tail                                     # Stream live logs
 npx bm logs:tail --fn bm_paymentsWebhookOnWrite      # Stream filtered live logs
 ```
 
-Both commands save output to `logs.log` in the project directory (overwritten on each run). `logs:read` saves raw JSON; `logs:tail` streams text.
+Both commands save output to `functions/logs.log` (overwritten on each run). `logs:read` saves raw JSON; `logs:tail` streams text.
 
 | Flag | Description | Default | Commands |
 |------|-------------|---------|----------|

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "backend-manager",
-  "version": "5.0.133",
+  "version": "5.0.135",
   "description": "Quick tools for developing Firebase functions",
   "main": "src/manager/index.js",
   "bin": {
@@ -18,11 +18,11 @@
     "node": "22"
   },
   "projectScripts": {
-    "start": "npx bm setup && npx bm serve",
-    "deploy": "npx bm setup && npx bm deploy",
-    "emulator": "npx bm setup && npx bm emulator",
-    "test": "npx bm setup && npx bm test",
-    "setup": "npx bm setup"
+    "start": "npx mgr setup && npx mgr serve",
+    "deploy": "npx mgr setup && npx mgr deploy",
+    "emulator": "npx mgr setup && npx mgr emulator",
+    "test": "npx mgr setup && npx mgr test",
+    "setup": "npx mgr setup"
   },
   "repository": {
     "type": "git",

package/src/cli/commands/emulator.js

@@ -62,7 +62,7 @@ class EmulatorCommand extends BaseCommand {
     const emulatorCommand = `BEM_TESTING=true firebase emulators:exec --only functions,firestore,auth,database,hosting,pubsub --ui "${command}"`;
 
     // Set up log file in the project directory
-    const logPath = path.join(projectDir, 'emulator.log');
+    const logPath = path.join(projectDir, 'functions', 'emulator.log');
     const logStream = fs.createWriteStream(logPath, { flags: 'w' });
     const stripAnsi = (str) => str.replace(/\x1B\[[0-9;]*[a-zA-Z]/g, '');
 

package/src/cli/commands/logs.js

@@ -73,7 +73,7 @@ class LogsCommand extends BaseCommand {
 
     // Set up log file in the project directory
     const projectDir = this.main.firebaseProjectPath;
-    const logPath = path.join(projectDir, 'logs.log');
+    const logPath = path.join(projectDir, 'functions', 'logs.log');
 
     this.log(chalk.gray(`  Filter: ${filter || '(none)'}`));
     this.log(chalk.gray(`  Limit: ${limit}`));
@@ -127,7 +127,7 @@ class LogsCommand extends BaseCommand {
 
     // Set up log file in the project directory
     const projectDir = this.main.firebaseProjectPath;
-    const logPath = path.join(projectDir, 'logs.log');
+    const logPath = path.join(projectDir, 'functions', 'logs.log');
     const logStream = fs.createWriteStream(logPath, { flags: 'w' });
 
     const filter = this.buildFilter(argv, { excludeTimestamp: true });

package/src/cli/commands/serve.js

@@ -1,4 +1,7 @@
 const BaseCommand = require('./base-command');
+const path = require('path');
+const fs = require('fs');
+const chalk = require('chalk');
 const powertools = require('node-powertools');
 const WatchCommand = require('./watch');
 
@@ -6,6 +9,7 @@ class ServeCommand extends BaseCommand {
   async execute() {
     const self = this.main;
     const port = self.argv.port || self.argv?._?.[1] || '5000';
+    const projectDir = self.firebaseProjectPath;
 
     // Start BEM watcher in background
     const watcher = new WatchCommand(self);
@@ -14,8 +18,44 @@ class ServeCommand extends BaseCommand {
     // Start Stripe webhook forwarding in background
     this.startStripeWebhookForwarding();
 
-    // Execute
-    await powertools.execute(`firebase serve --port ${port}`, { log: true });
+    // Set up log file in the project directory
+    const logPath = path.join(projectDir, 'functions', 'serve.log');
+    const logStream = fs.createWriteStream(logPath, { flags: 'w' });
+    const stripAnsi = (str) => str.replace(/\x1B\[[0-9;]*[a-zA-Z]/g, '');
+
+    this.log(chalk.gray(`  Logs saving to: ${logPath}\n`));
+
+    // Execute with tee to log file
+    try {
+      await powertools.execute(`firebase serve --port ${port}`, {
+        log: false,
+        cwd: projectDir,
+        config: {
+          stdio: ['inherit', 'pipe', 'pipe'],
+          env: { ...process.env, FORCE_COLOR: '1' },
+        },
+      }, (child) => {
+        // Tee stdout to both console and log file (strip ANSI codes for clean log)
+        child.stdout.on('data', (data) => {
+          process.stdout.write(data);
+          logStream.write(stripAnsi(data.toString()));
+        });
+
+        // Tee stderr to both console and log file (strip ANSI codes for clean log)
+        child.stderr.on('data', (data) => {
+          process.stderr.write(data);
+          logStream.write(stripAnsi(data.toString()));
+        });
+
+        // Clean up log stream when child exits
+        child.on('close', () => {
+          logStream.end();
+        });
+      });
+    } catch (error) {
+      // User pressed Ctrl+C - this is expected
+      this.log(chalk.gray('\n  Server stopped.\n'));
+    }
   }
 }
 

package/src/cli/commands/test.js

@@ -185,7 +185,7 @@ class TestCommand extends BaseCommand {
     this.log(chalk.gray(`  UI: http://127.0.0.1:${emulatorPorts.ui}`));
 
     // Set up log file in the project directory
-    const logPath = path.join(projectDir, 'test.log');
+    const logPath = path.join(projectDir, 'functions', 'test.log');
     const logStream = fs.createWriteStream(logPath, { flags: 'w' });
     const stripAnsi = (str) => str.replace(/\x1B\[[0-9;]*[a-zA-Z]/g, '');
 

package/src/manager/routes/admin/post/post.js

@@ -1,6 +1,7 @@
 /**
  * POST /admin/post - Create blog post
  * Admin/blogger endpoint to create blog posts via GitHub
+ * Uses Git Trees API to commit all files (images + post) in a single commit
  */
 const moment = require('moment');
 const jetpack = require('fs-jetpack');
@@ -15,7 +16,6 @@ const IMAGE_PATH_SRC = `src/assets/images/blog/post-{id}/`;
 const IMAGE_REGEX = /(?:!\[(.*?)\]\((.*?)\))/img;
 
 module.exports = async ({ assistant, Manager, user, settings, analytics }) => {
-  const fetch = Manager.require('wonderful-fetch');
 
   // Require authentication
   if (!user.authenticated) {
@@ -92,30 +92,47 @@ module.exports = async ({ assistant, Manager, user, settings, analytics }) => {
 
   assistant.log('main(): Creating post...', settings);
 
-  // Extract all images
-  const imageResult = await extractImages(assistant, octokit, settings).catch(e => e);
-  if (imageResult instanceof Error) {
-    return assistant.respond(imageResult.message, { code: 400 });
+  // Download all images and collect file data
+  const imageFiles = await downloadImages(assistant, settings).catch(e => e);
+  if (imageFiles instanceof Error) {
+    return assistant.respond(imageFiles.message, { code: 400 });
   }
 
   // Rewrite body to use @post/ prefix for extracted images
-  for (const { originalUrl, localFilename } of imageResult) {
-    settings.body = settings.body.split(originalUrl).join(`@post/${localFilename}`);
+  for (const file of imageFiles) {
+    if (file.originalUrl) {
+      settings.body = settings.body.split(file.originalUrl).join(`@post/${file.filename}`);
+    }
   }
 
-  // Set defaults
+  // Generate post content from template
   const formattedContent = powertools.template(
     POST_TEMPLATE,
     formatClone(settings),
   );
 
-  // Upload post
-  const uploadResult = await uploadPost(assistant, octokit, settings, formattedContent).catch(e => e);
-  if (uploadResult instanceof Error) {
-    return assistant.respond(uploadResult.message, { code: 500 });
+  // Build post file entry
+  const postFilename = `${settings.path}/${settings.date}-${settings.url}.md`;
+  const allFiles = [
+    ...imageFiles.map(img => ({
+      path: img.githubPath,
+      content: img.base64,
+      encoding: 'base64',
+    })),
+    {
+      path: postFilename,
+      content: Buffer.from(formattedContent).toString('base64'),
+      encoding: 'base64',
+    },
+  ];
+
+  // Commit all files in a single commit
+  const commitResult = await commitAll(assistant, octokit, settings, allFiles).catch(e => e);
+  if (commitResult instanceof Error) {
+    return assistant.respond(commitResult.message, { code: 500 });
   }
 
-  assistant.log('main(): uploadPost', uploadResult);
+  assistant.log('main(): commitAll', commitResult);
 
   // Track analytics
   analytics.event('admin/post', { action: 'create' });
@@ -123,9 +140,10 @@ module.exports = async ({ assistant, Manager, user, settings, analytics }) => {
   return assistant.respond(settings);
 };
 
-// Helper: Extract and upload images
-async function extractImages(assistant, octokit, settings) {
-  const urlMap = [];
+// Helper: Download all images and return file data (no GitHub uploads)
+async function downloadImages(assistant, settings) {
+  const files = [];
+  const assetsPath = powertools.template(IMAGE_PATH_SRC, settings);
 
   const matches = settings.body.matchAll(IMAGE_REGEX);
   const images = Array.from(matches).map(match => ({
@@ -141,10 +159,10 @@ async function extractImages(assistant, octokit, settings) {
     header: true,
   });
 
-  assistant.log('extractImages(): images', images);
+  assistant.log('downloadImages(): images', images);
 
   if (!images.length) {
-    return urlMap;
+    return files;
   }
 
   for (let index = 0; index < images.length; index++) {
@@ -153,38 +171,29 @@ async function extractImages(assistant, octokit, settings) {
     // Download image
     const download = await downloadImage(assistant, image.src, image.alt).catch(e => e);
 
-    assistant.log('extractImages(): download', download);
+    assistant.log('downloadImages(): download', download);
 
     if (download instanceof Error) {
       if (image.header) {
         throw download;
       } else {
-        assistant.warn('extractImages(): Skipping NON-HEADER image download due to error', download);
+        assistant.warn('downloadImages(): Skipping NON-HEADER image download due to error', download);
         continue;
       }
     }
 
-    // Upload image
-    const upload = await uploadImage(assistant, octokit, settings, download).catch(e => e);
+    // Read file content as base64
+    const base64 = jetpack.read(download.path, 'buffer').toString('base64');
 
-    assistant.log('extractImages(): upload', upload);
-
-    if (upload instanceof Error) {
-      if (image.header) {
-        throw upload;
-      } else {
-        assistant.warn('extractImages(): Skipping NON-HEADER image upload due to error', upload);
-        continue;
-      }
-    }
-
-    // Track successfully uploaded non-header images for body rewriting
-    if (!image.header) {
-      urlMap.push({ originalUrl: image.src, localFilename: download.filename });
-    }
+    files.push({
+      githubPath: `${assetsPath}${download.filename}`,
+      filename: download.filename,
+      base64: base64,
+      originalUrl: image.header ? null : image.src,
+    });
   }
 
-  return urlMap;
+  return files;
 }
 
 // Helper: Download image
@@ -211,82 +220,95 @@ async function downloadImage(assistant, src, alt) {
   return result;
 }
 
-// Helper: Upload image to GitHub
-async function uploadImage(assistant, octokit, settings, image) {
-  const filepath = image.path;
-  const filename = image.filename;
-  const assetsPath = powertools.template(IMAGE_PATH_SRC, settings);
+// Helper: Commit all files (images + post) in a single commit using Git Trees API
+async function commitAll(assistant, octokit, settings, files) {
   const owner = settings.githubUser;
   const repo = settings.githubRepo;
 
-  assistant.log('uploadImage(): image', image);
-  assistant.log('uploadImage(): path', `${assetsPath}${filename}`);
+  assistant.log('commitAll(): Committing', files.length, 'files');
 
-  // Get existing image
-  const existing = await octokit.rest.repos.getContent({
+  // Get the latest commit SHA on the default branch
+  const refResult = await octokit.rest.git.getRef({
     owner: owner,
     repo: repo,
-    path: `${assetsPath}${filename}`,
-  }).catch(e => e);
+    ref: 'heads/master',
+  }).catch(() => {
+    // Try 'main' if 'master' fails
+    return octokit.rest.git.getRef({
+      owner: owner,
+      repo: repo,
+      ref: 'heads/main',
+    });
+  });
 
-  assistant.log('uploadImage(): Existing', existing);
+  const latestCommitSha = refResult.data.object.sha;
+  const branch = refResult.data.ref;
 
-  if (existing instanceof Error && existing?.status !== 404) {
-    throw existing;
-  }
+  assistant.log('commitAll(): Latest commit', latestCommitSha, 'on', branch);
 
-  // Upload image
-  const result = await octokit.rest.repos.createOrUpdateFileContents({
+  // Get the tree SHA of the latest commit
+  const commitResult = await octokit.rest.git.getCommit({
     owner: owner,
     repo: repo,
-    path: `${assetsPath}${filename}`,
-    sha: existing?.data?.sha || undefined,
-    message: `📦 admin/post:upload-image ${filename}`,
-    content: jetpack.read(filepath, 'buffer').toString('base64'),
+    commit_sha: latestCommitSha,
   });
 
-  assistant.log('uploadImage(): Result', result);
+  const baseTreeSha = commitResult.data.tree.sha;
 
-  return result;
-}
+  // Create blobs for each file
+  const treeItems = [];
 
-// Helper: Upload post to GitHub
-async function uploadPost(assistant, octokit, settings, content) {
-  const filename = `${settings.path}/${settings.date}-${settings.url}.md`;
-  const owner = settings.githubUser;
-  const repo = settings.githubRepo;
+  for (const file of files) {
+    const blob = await octokit.rest.git.createBlob({
+      owner: owner,
+      repo: repo,
+      content: file.content,
+      encoding: file.encoding,
+    });
+
+    assistant.log('commitAll(): Created blob for', file.path, blob.data.sha);
 
-  assistant.log('uploadPost(): filename', filename);
+    treeItems.push({
+      path: file.path,
+      mode: '100644',
+      type: 'blob',
+      sha: blob.data.sha,
+    });
+  }
 
-  // Get existing post
-  const existing = await octokit.rest.repos.getContent({
+  // Create a new tree with all files
+  const newTree = await octokit.rest.git.createTree({
     owner: owner,
     repo: repo,
-    path: filename,
-  }).catch(e => e);
+    base_tree: baseTreeSha,
+    tree: treeItems,
+  });
 
-  assistant.log('uploadPost(): Existing', existing);
+  assistant.log('commitAll(): Created tree', newTree.data.sha);
 
-  if (existing instanceof Error && existing?.status !== 404) {
-    throw existing;
-  }
+  // Create the commit
+  const postPath = files[files.length - 1].path;
+  const newCommit = await octokit.rest.git.createCommit({
+    owner: owner,
+    repo: repo,
+    message: `📦 admin/post:create ${postPath}`,
+    tree: newTree.data.sha,
+    parents: [latestCommitSha],
+  });
 
-  // Wait for GitHub to process images
-  await powertools.wait(30000);
+  assistant.log('commitAll(): Created commit', newCommit.data.sha);
 
-  // Upload post
-  const result = await octokit.rest.repos.createOrUpdateFileContents({
+  // Update the branch ref to point to the new commit
+  const updateResult = await octokit.rest.git.updateRef({
     owner: owner,
     repo: repo,
-    path: filename,
-    sha: existing?.data?.sha || undefined,
-    message: `📦 admin/post:upload-post ${filename}`,
-    content: Buffer.from(content).toString('base64'),
+    ref: branch.replace('refs/', ''),
+    sha: newCommit.data.sha,
   });
 
-  assistant.log('uploadPost(): Result', result);
+  assistant.log('commitAll(): Updated ref', updateResult.data.object.sha);
 
-  return result;
+  return updateResult;
 }
 
 // Helper: Format clone for templating