@npmcli/template-oss 4.1.1 → 4.2.0

package/lib/util/template.js

@@ -3,35 +3,37 @@ const { basename, extname, join } = require('path')
 const fs = require('fs')
 const DELETE = '__DELETE__'
 
-const partialName = (s) =>
-  basename(s, extname(s)).replace(/-([a-z])/g, (_, g) => g.toUpperCase())
+const partialName = (s) => basename(s, extname(s)) // remove extension
+  .replace(/^_/, '') // remove leading underscore
+  .replace(/-([a-z])/g, (_, g) => g.toUpperCase()) // camelcase
 
-const setupHandlebars = (partialsDir) => {
+const setupHandlebars = (...partialDirs) => {
   Handlebars.registerHelper('obj', ({ hash }) => hash)
   Handlebars.registerHelper('json', (c) => JSON.stringify(c))
   Handlebars.registerHelper('del', () => JSON.stringify(DELETE))
 
-  // Load all content files as camelcase partial names
-  for (const f of fs.readdirSync(join(partialsDir))) {
-    Handlebars.registerPartial(
-      partialName(f),
-      fs.readFileSync(join(partialsDir, f)).toString()
-    )
+  // Load all files as camelcase partial names.
+  // all other content dirs only get special underscore leading
+  // files as partials. this prevents recursion loops when overwriting
+  // a filename to use as a enw file
+  let isBase = true
+  for (const dir of partialDirs) {
+    for (const f of fs.readdirSync(dir)) {
+      if (f.startsWith('_') || isBase) {
+        Handlebars.registerPartial(
+          partialName(f),
+          fs.readFileSync(join(dir, f)).toString()
+        )
+      }
+    }
+    isBase = false
   }
 }
 
-const cache = new Map()
-
 const template = (str, { config, ...options }) => {
-  if (cache.size === 0) {
-    setupHandlebars(config.sourceDir)
-  }
+  setupHandlebars(...config.__PARTIAL_DIRS__)
 
-  let t = cache.get(str)
-  if (t == null) {
-    t = Handlebars.compile(str, { strict: true })
-    cache.set(str, t)
-  }
+  const t = Handlebars.compile(str, { strict: true })
 
   // merge in config as top level data in templates
   return t({ ...options, ...config })

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@npmcli/template-oss",
-  "version": "4.1.1",
+  "version": "4.2.0",
   "description": "templated files used in npm CLI team oss projects",
   "main": "lib/content/index.js",
   "bin": {
@@ -32,6 +32,7 @@
     "@actions/core": "^1.9.1",
     "@commitlint/cli": "^17.1.1",
     "@commitlint/config-conventional": "^17.1.0",
+    "@isaacs/string-locale-compare": "^1.1.0",
     "@npmcli/fs": "^2.0.1",
     "@npmcli/git": "^3.0.0",
     "@npmcli/map-workspaces": "^2.0.2",
@@ -41,6 +42,7 @@
     "handlebars": "^4.7.7",
     "hosted-git-info": "^5.0.0",
     "json-parse-even-better-errors": "^2.3.1",
+    "just-deep-map-values": "^1.1.1",
     "just-diff": "^5.0.1",
     "lodash": "^4.17.21",
     "npm-package-arg": "^9.0.1",
@@ -59,7 +61,11 @@
     "tap": "^16.0.0"
   },
   "tap": {
-    "timeout": 600
+    "timeout": 600,
+    "nyc-arg": [
+      "--exclude",
+      "tap-snapshots/**"
+    ]
   },
   "templateOSS": {
     "//@npmcli/template-oss": "This file is partially managed by @npmcli/template-oss. Edits may be overwritten."

package/lib/content/release-test.yml

@@ -1,46 +0,0 @@
-name: Release
-
-on:
-  workflow_call:
-    inputs:
-      ref:
-        required: true
-        type: string
-
-jobs:
-  lint-all:
-    runs-on: ubuntu-latest
-    steps:
-      {{> setupGit with=(obj ref="${{ inputs.ref }}")}}
-      {{> setupNode}}
-      {{> setupDeps}}
-      - run: npm run lint --if-present --workspaces --include-workspace-root
-
-  test-all:
-    strategy:
-      fail-fast: false
-      matrix:
-        node-version:
-          {{#each ciVersions}}
-          - {{.}}
-          {{/each}}
-        platform:
-          - os: ubuntu-latest
-            shell: bash
-          - os: macos-latest
-            shell: bash
-          {{#if windowsCI}}
-          - os: windows-latest
-            shell: cmd
-          {{/if}}
-    runs-on: $\{{ matrix.platform.os }}
-    defaults:
-      run:
-        shell: $\{{ matrix.platform.shell }}
-    steps:
-      {{> setupGit with=(obj ref="${{ inputs.ref }}")}}
-      {{> setupNode useMatrix=true}}
-      {{> setupDeps}}
-      - name: add tap problem matcher
-        run: echo "::add-matcher::.github/matchers/tap.json"
-      - run: npm run test --if-present --workspaces --include-workspace-root

package/lib/content/setup-deps.yml

@@ -1 +0,0 @@
-- run: npm i --ignore-scripts --no-audit --no-fund {{~#if flags}} {{flags}}{{/if}}