poops 1.9.1 → 1.9.3

package/README.md

@@ -110,6 +110,7 @@ or pass a custom config. This is useful when you have multiple environments:
 | `--port <number>`            | `-p`  | Specify the server port, overrides config            |
 | `--livereload-port <number>` | `-l`  | Specify the livereload port, overrides config        |
 | `--base-url <path>`          | `-u`  | Set the base URL prefix for markup, overrides config |
+| `--quiet`                    | `-q`  | Hide the header and the server/livereload info lines |
 
 The `--base-url` flag is particularly useful for CI/CD pipelines where the deploy path may differ per environment:
 
@@ -117,6 +118,14 @@ The `--base-url` flag is particularly useful for CI/CD pipelines where the deplo
 poops --build --base-url /blog
 ```
 
+The `--quiet` flag drops the `💩 Poops — vX.Y.Z` header (and its terminal bell) plus the `Local server` / `Network` / `LiveReload` lines. Handy when you run several Poops instances side by side and only want to see which one is compiling:
+
+```bash
+poops -q & poops -q -c site/poops.json
+```
+
+Build logs, warnings and errors are unaffected — `--quiet` only removes the banner.
+
 If you have installed Poops locally you can run it with `npx poops` or `npx 💩` or add a script to your `package.json`:
 
 ```json

package/lib/server.js

@@ -63,16 +63,25 @@ export function parseRange(header, size) {
 }
 
 export function createStaticHandler(base) {
+  // Drop any trailing separator — `serve.base: "/"` resolves to `<cwd>/`, and the
+  // containment check below would then compare against `<cwd>//` and reject everything
+  base = path.resolve(base)
   const notFoundPage = path.join(base, '404.html')
 
   const notFound = (res) => {
     res.statusCode = 404
-    if (pathExists(notFoundPage)) {
-      res.setHeader('Content-Type', 'text/html; charset=utf-8')
-      fs.createReadStream(notFoundPage).on('error', () => res.destroy()).pipe(res)
-    } else {
-      res.end('Not Found')
+    if (!pathExists(notFoundPage)) return res.end('Not Found')
+    res.setHeader('Content-Type', 'text/html; charset=utf-8')
+    // 404.html lives at the site root but is served at any depth — its relative
+    // asset paths would resolve against /a/ for /a/b, so pin them with <base>
+    let html
+    try {
+      html = fs.readFileSync(notFoundPage, 'utf8')
+    } catch {
+      return res.end('Not Found')
     }
+    if (!/<base[\s>]/i.test(html)) html = html.replace(/<head([^>]*)>/i, '<head$1><base href="/">')
+    res.end(html)
   }
 
   return (req, res) => {
@@ -100,7 +109,13 @@ export function createStaticHandler(base) {
     try {
       stat = fs.statSync(filePath)
     } catch {
-      return notFound(res)
+      // GitHub Pages-style extensionless URLs: /a/b serves a/b.html, URL unchanged
+      try {
+        stat = fs.statSync(filePath + '.html')
+        filePath += '.html'
+      } catch {
+        return notFound(res)
+      }
     }
 
     if (stat.isDirectory()) {

package/lib/utils/minify.js

@@ -37,5 +37,7 @@ export default async function minifyToFile({ outfilePath, loader, code, banner,
     console.error(err)
   }
 
-  if (options.justMinified) fs.unlinkSync(outfilePath)
+  // The unminified file only exists on disk when a compiler wrote it there —
+  // watch rebuilds pass `code` in memory and there is nothing to delete.
+  if (options.justMinified && pathExists(outfilePath)) fs.unlinkSync(outfilePath)
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "poops",
   "description": "Straightforward, no-bullshit bundler for the web.",
-  "version": "1.9.1",
+  "version": "1.9.3",
   "license": "MIT",
   "type": "module",
   "main": "poops.js",

package/poops.js

@@ -30,6 +30,7 @@ const cli = new Argoyle(pkg.version)
   .option('port', { short: 'p', value: '<number>', description: 'Specify the port for the server, overrides the config file' })
   .option('livereload-port', { short: 'l', value: '<number>', description: 'Specify the port for the livereload server, overrides the config file' })
   .option('base-url', { short: 'u', value: '<path>', description: 'Set the base URL prefix for markup, overrides the config file' })
+  .option('quiet', { short: 'q', description: 'Hide the header and the server/livereload info lines' })
 
 let flags, positionals
 try {
@@ -44,6 +45,7 @@ const defaultConfigPath = flags.config || positionals[0] || 'poops.json'
 const overridePort = flags.port
 const overrideLivereloadPort = flags['livereload-port']
 const overrideBaseURL = flags['base-url']
+const quiet = flags.quiet // hides the header and the address lines only — build logs, warnings and errors still print
 
 let configPath = path.join(cwd, defaultConfigPath)
 if (!pathExists(configPath)) configPath = path.join(cwd, '💩.json') // the canonical alternative config name
@@ -107,7 +109,7 @@ const hook = (stage) => runExec(config, cwd, stage)
 function setupLiveReloadServer(config) {
   if (!config.livereload) return
   liveReloadServer = livereload.createServer({ port: config.livereload_port })
-  styledLog(`🔃 {dim}LiveReload  :{/} ${liveReloadServer.config.port}`)
+  if (!quiet) styledLog(`🔃 {dim}LiveReload  :{/} ${liveReloadServer.config.port}`)
   console.log()
 }
 
@@ -296,8 +298,10 @@ async function poops() {
 }
 
 // CLI Header
-const title = `💩 Poops — v${pkg.version}`
-styledLog(`\n{#8b4513}${title}\n${title.replace(/./g, '-')}{/}{bell}\n`)
+if (!quiet) {
+  const title = `💩 Poops — v${pkg.version}`
+  styledLog(`\n{#8b4513}${title}\n${title.replace(/./g, '-')}{/}{bell}\n`)
+}
 
 // Check if poops.json exists
 if (!pathExists(configPath)) {
@@ -391,9 +395,11 @@ async function startServer() {
 
   // eslint-disable-next-line @stylistic/space-before-function-paren
   http.createServer(createStaticHandler(base)).listen(parseInt(port), '0.0.0.0', async () => {
-    console.log()
-    styledLog(`🏠 {dim}Local server:{/} {underline|http://localhost:${port}}`)
-    styledLog(`🛜 {dim} Network     :{/} {underline|http://${getLocalIP()}:${port}}`)
+    if (!quiet) {
+      console.log()
+      styledLog(`🏠 {dim}Local server:{/} {underline|http://localhost:${port}}`)
+      styledLog(`🛜 {dim} Network     :{/} {underline|http://${getLocalIP()}:${port}}`)
+    }
     setupLiveReloadServer(config)
   })
 }