@zeropress/build-pages 0.5.2 → 0.5.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zeropress/build-pages",
-  "version": "0.5.2",
+  "version": "0.5.5",
   "description": "ZeroPress Markdown build action and CLI for static hosting",
   "type": "module",
   "license": "MIT",

package/src/action.js

@@ -8,7 +8,8 @@ const options = {
   config: input('config'),
   siteUrl: input('site-url'),
   skipUntitledMarkdown: booleanInput('skip-untitled-markdown', false),
-  checkLinks: booleanInput('check-links', true),
+  skipLinkCheck: booleanInput('skip-link-check', false),
+  copyMarkdownSource: falseOnlyInput('copy-markdown-source'),
 };
 
 try {
@@ -30,3 +31,7 @@ function booleanInput(name, fallback) {
   }
   return value.toLowerCase() === 'true';
 }
+
+function falseOnlyInput(name) {
+  return input(name).toLowerCase() !== 'false';
+}

package/src/index.js

@@ -7,7 +7,9 @@ import { checkInternalLinks } from './check-links.js';
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 const packageDir = path.resolve(__dirname, '..');
-const prebuildScript = path.join(packageDir, 'src', 'prebuild.js');
+const prebuildScript = __dirname === path.join(packageDir, 'dist')
+  ? path.join(__dirname, 'prebuild.js')
+  : path.join(packageDir, 'src', 'prebuild.js');
 const PREVIEW_DATA_PATH = '.zeropress/preview-data.json';
 const STAGING_DIR = '.zeropress/public-assets';
 const DEFAULT_THEME = 'docs';
@@ -35,6 +37,7 @@ export async function runCli(argv = process.argv.slice(2)) {
 
 export async function runBuildPages(options) {
   const cwd = path.resolve(options.cwd || process.cwd());
+  const copyMarkdownSource = options.copyMarkdownSource !== false;
   const sourceDir = path.resolve(cwd, options.source);
   const destinationDir = path.resolve(cwd, options.destination);
   const generatedDir = path.join(cwd, '.zeropress');
@@ -42,6 +45,13 @@ export async function runBuildPages(options) {
   const previewDataPath = path.join(cwd, PREVIEW_DATA_PATH);
   const themeDir = resolveThemeDir(cwd, options);
 
+  assertBuildPagesPathLayout({
+    cwd,
+    sourceDir,
+    destinationDir,
+    themeDir,
+    generatedDir,
+  });
   await assertDirectory(sourceDir, 'Source directory');
   await fs.rm(generatedDir, { recursive: true, force: true });
   await fs.mkdir(generatedDir, { recursive: true });
@@ -51,6 +61,7 @@ export async function runBuildPages(options) {
     ZEROPRESS_BUILD_PAGES_SOURCE: sourceDir,
     ZEROPRESS_PUBLIC_DIR: sourceDir,
     ZEROPRESS_SKIP_UNTITLED_MARKDOWN: String(Boolean(options.skipUntitledMarkdown)),
+    ZEROPRESS_COPY_MARKDOWN_SOURCE: String(copyMarkdownSource),
   };
   if (options.config) {
     env.ZEROPRESS_BUILD_PAGES_CONFIG = path.resolve(cwd, options.config);
@@ -76,6 +87,7 @@ export async function runBuildPages(options) {
   await fs.mkdir(stagingDir, { recursive: true });
   await copyPublicStaging(sourceDir, stagingDir, {
     excludePaths: [destinationDir, themeDir, generatedDir],
+    copyMarkdownSource,
   });
 
   const previousPublicDir = process.env.ZEROPRESS_PUBLIC_DIR;
@@ -93,7 +105,7 @@ export async function runBuildPages(options) {
     }
   }
 
-  if (options.checkLinks) {
+  if (!options.skipLinkCheck) {
     const result = await checkInternalLinks(destinationDir);
     if (result.brokenLinks.length) {
       console.warn('Warning: broken internal links found:');
@@ -105,7 +117,7 @@ export async function runBuildPages(options) {
   }
 }
 
-export function parseArgs(argv, env = process.env) {
+export function parseArgs(argv) {
   const flags = {};
 
   for (let index = 0; index < argv.length; index += 1) {
@@ -114,8 +126,12 @@ export function parseArgs(argv, env = process.env) {
       flags.skipUntitledMarkdown = true;
       continue;
     }
-    if (arg === '--no-check-links') {
-      flags.checkLinks = false;
+    if (arg === '--skip-link-check') {
+      flags.skipLinkCheck = true;
+      continue;
+    }
+    if (arg === '--no-copy-markdown-source') {
+      flags.copyMarkdownSource = false;
       continue;
     }
 
@@ -140,24 +156,25 @@ export function parseArgs(argv, env = process.env) {
     throw new Error(`Invalid arguments: unknown option ${arg}`);
   }
 
-  const source = flags.source || env.ZEROPRESS_PUBLIC_DIR || '';
-  const destination = flags.destination || env.ZEROPRESS_OUT_DIR || '';
+  const source = flags.source || '';
+  const destination = flags.destination || '';
   if (!source) {
-    throw new Error('Invalid arguments: --source <dir> is required. You can also set ZEROPRESS_PUBLIC_DIR.');
+    throw new Error('Invalid arguments: --source <dir> is required.');
   }
   if (!destination) {
-    throw new Error('Invalid arguments: --destination <dir> is required. You can also set ZEROPRESS_OUT_DIR.');
+    throw new Error('Invalid arguments: --destination <dir> is required.');
   }
 
   return {
     source,
     destination,
     theme: flags.theme || DEFAULT_THEME,
-    themePath: flags['theme-path'] || env.ZEROPRESS_THEME_DIR || '',
-    config: flags.config || env.ZEROPRESS_BUILD_PAGES_CONFIG || '',
-    siteUrl: flags['site-url'] || env.ZEROPRESS_SITE_URL || '',
-    skipUntitledMarkdown: flags.skipUntitledMarkdown ?? env.ZEROPRESS_SKIP_UNTITLED_MARKDOWN === 'true',
-    checkLinks: flags.checkLinks ?? true,
+    themePath: flags['theme-path'] || '',
+    config: flags.config || '',
+    siteUrl: flags['site-url'] || '',
+    skipUntitledMarkdown: flags.skipUntitledMarkdown === true,
+    skipLinkCheck: flags.skipLinkCheck === true,
+    copyMarkdownSource: flags.copyMarkdownSource !== false,
   };
 }
 
@@ -168,14 +185,15 @@ Usage:
   zeropress-build-pages [options]
 
 Options:
-  --source <dir>                Source directory (required, or ZEROPRESS_PUBLIC_DIR)
-  --destination <dir>           Output directory (required, or ZEROPRESS_OUT_DIR)
+  --source <dir>                Dedicated source directory (required)
+  --destination <dir>           Output directory (required)
   --theme docs                  Bundled theme name (default: docs)
   --theme-path <dir>            Custom ZeroPress theme directory
   --config <path>               Config file (default: <source>/.zeropress/config.json)
   --site-url <url>              Canonical site URL override
   --skip-untitled-markdown      Skip Markdown files without a page title
-  --no-check-links              Skip internal link checking
+  --skip-link-check             Skip internal link checking
+  --no-copy-markdown-source     Do not copy original Markdown files to output
   --help, -h                    Show help
   --version, -v                 Show version`);
 }
@@ -205,6 +223,32 @@ async function assertDirectory(dir, label) {
   }
 }
 
+function assertBuildPagesPathLayout({ cwd, sourceDir, destinationDir, themeDir, generatedDir }) {
+  if (samePath(sourceDir, cwd)) {
+    throw new Error(
+      'Source directory must be a dedicated content directory, not the current working directory. '
+      + `Received: ${formatPath(cwd, sourceDir)}`,
+    );
+  }
+
+  assertNoPathOverlap(cwd, 'Source directory', sourceDir, 'internal .zeropress working directory', generatedDir);
+  assertNoPathOverlap(cwd, 'Destination directory', destinationDir, 'internal .zeropress working directory', generatedDir);
+  assertNoPathOverlap(cwd, 'Theme directory', themeDir, 'internal .zeropress working directory', generatedDir);
+  assertNoPathOverlap(cwd, 'Source directory', sourceDir, 'destination directory', destinationDir);
+  assertNoPathOverlap(cwd, 'Source directory', sourceDir, 'theme directory', themeDir);
+}
+
+function assertNoPathOverlap(cwd, firstLabel, firstPath, secondLabel, secondPath) {
+  if (!pathsOverlap(firstPath, secondPath)) {
+    return;
+  }
+  throw new Error(
+    `${firstLabel} must not overlap the ${secondLabel}. `
+    + `${firstLabel}: ${formatPath(cwd, firstPath)}; `
+    + `${secondLabel}: ${formatPath(cwd, secondPath)}`,
+  );
+}
+
 async function copyPublicStaging(sourceDir, targetDir, options) {
   const entries = await fs.readdir(sourceDir, { withFileTypes: true });
 
@@ -229,6 +273,10 @@ async function copyPublicStaging(sourceDir, targetDir, options) {
       continue;
     }
 
+    if (options.copyMarkdownSource === false && entry.name.toLowerCase().endsWith('.md')) {
+      continue;
+    }
+
     await fs.mkdir(path.dirname(targetPath), { recursive: true });
     await fs.copyFile(sourcePath, targetPath);
   }
@@ -256,6 +304,10 @@ function pathsOverlap(firstPath, secondPath) {
   return first === second || isPathInside(first, second) || isPathInside(second, first);
 }
 
+function samePath(firstPath, secondPath) {
+  return path.resolve(firstPath) === path.resolve(secondPath);
+}
+
 function isPathInside(parentPath, childPath) {
   const relativePath = path.relative(parentPath, childPath);
   return Boolean(relativePath) && !relativePath.startsWith('..') && !path.isAbsolute(relativePath);

package/src/prebuild.js

@@ -5,7 +5,7 @@ import matter from 'gray-matter';
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 const rootDir = process.cwd();
-const sourceDir = resolveEnvPath(['ZEROPRESS_BUILD_PAGES_SOURCE', 'ZEROPRESS_PUBLIC_DIR'], 'docs');
+const sourceDir = resolveEnvPath(['ZEROPRESS_BUILD_PAGES_SOURCE'], 'docs');
 const defaultConfigPath = path.join(sourceDir, '.zeropress', 'config.json');
 const configPath = resolveOptionalEnvPath(['ZEROPRESS_BUILD_PAGES_CONFIG'], defaultConfigPath);
 const outDir = path.join(rootDir, '.zeropress');
@@ -13,6 +13,7 @@ const buildPagesConfigPath = path.join(outDir, 'build-pages-config.json');
 const previewDataPath = path.join(outDir, 'preview-data.json');
 const buildReportPath = path.join(outDir, 'build-report.json');
 const skipUntitledMarkdown = readBooleanEnv('ZEROPRESS_SKIP_UNTITLED_MARKDOWN');
+const copyMarkdownSource = readBooleanEnv('ZEROPRESS_COPY_MARKDOWN_SOURCE', true);
 const FRONT_PAGE_TYPES = new Set(['theme_index', 'markdown', 'html']);
 const BUILD_PAGES_CONFIG_SCHEMA_URL = 'https://zeropress.dev/schemas/zeropress-build-pages.config.v0.1.schema.json';
 const PREVIEW_DATA_SCHEMA_URL = 'https://zeropress.dev/schemas/preview-data.v0.5.schema.json';
@@ -96,7 +97,7 @@ async function main() {
     path: route.path,
     meta: {
       ...frontMatter.meta,
-      source_markdown_url: buildSourceMarkdownUrl(sourcePath),
+      ...(copyMarkdownSource ? { source_markdown_url: buildSourceMarkdownUrl(sourcePath) } : {}),
     },
     content: rewriteMarkdownLinks(bodyMarkdown, sourcePath, routeBySourcePath),
     document_type: 'markdown',
@@ -733,6 +734,7 @@ function buildPrebuildReport({
     preview_data_path: formatSourcePath(previewDataPath),
     report_path: formatSourcePath(buildReportPath),
     skip_untitled_markdown: skipUntitledMarkdown,
+    copy_markdown_source: copyMarkdownSource,
     markdown: {
       discovered: sourceFiles.length,
       generated_pages: pageInputs.length,
@@ -1305,8 +1307,12 @@ function readConfigInteger(value, fallback) {
   return Number.isInteger(value) && value > 0 ? value : fallback;
 }
 
-function readBooleanEnv(name) {
-  return process.env[name]?.trim().toLowerCase() === 'true';
+function readBooleanEnv(name, fallback = false) {
+  const value = process.env[name]?.trim();
+  if (!value) {
+    return fallback;
+  }
+  return value.toLowerCase() === 'true';
 }
 
 function resolveEnvPath(names, fallback) {