@leverege/build-tools 2.102.0 → 2.104.0

package/.github/dependabot.yml

@@ -0,0 +1,7 @@
+version: 2
+updates:
+  - package-ecosystem: "npm"
+    directory: "/"
+    schedule:
+      interval: "weekly"
+    open-pull-requests-limit: 10

package/.vscode/launch.json

@@ -58,6 +58,16 @@
       "runtimeExecutable": "node",
       "console": "integratedTerminal"
     },
+    {
+      "name": "chart-to-registry rsrc-srvr",
+      "type": "node",
+      "request": "launch",
+      "program": "${workspaceFolder}/src/chart-to-registry.mjs",
+      "args": [],
+      "cwd": "/Users/john/Work/leverege/cicd/@plt/resource-server",
+      "runtimeExecutable": "node",
+      "console": "integratedTerminal"
+    },
     {
       "name": "ingestor in monorepo",
       "type": "node",

package/README.md

@@ -16,7 +16,7 @@ Most tools assume the following are installed and configured:
 - `kubectl` (with a valid cluster context)
 - `helm` (v3+)
 - `fzf` (for interactive pod selection in `k8x` and `klog`)
-- Node.js 22+
+- Node.js 22+ - prefer 24
 
 ---
 
@@ -85,13 +85,15 @@ init-my-chart [--dry-run] [--debug]
 ```
 
 **`chart-to-registry`** — Pushes Helm charts to GCP Artifact Registry with version
-checking and dependency resolution.
+checking and dependency resolution. Warns if `appVersion` in Chart.yaml does not start
+with `v` (bypass with `--ignore-appv`).
 ```bash
-chart-to-registry --chart <dir> [--project <id>] [--repository <name>] [--force] [--dry-run]
+chart-to-registry --chart <dir> [--project <id>] [--repository <name>] [--force] [--dry-run] [--ignore-appv]
 ```
 
 **`push-my-chart`** — Publishes Helm charts to both GCS Chartmuseum (legacy) and
-GCP Artifact Registry.
+GCP Artifact Registry. Displays chart name, version, and appVersion before pushing;
+warns if `appVersion` does not start with `v`.
 ```bash
 push-my-chart [--chart <dir>] [--no-registry]
 ```
@@ -226,11 +228,20 @@ Manager, updating `.npmrc` / `.yarnrc.yml`. Skips refresh if the token is less t
 refresh-npm-token [--project <id>] [--force] [--dry-run]
 ```
 
-**`pkgck`** — Checks for common package configuration issues (babel, eslint, circleci,
-dockerfile, etc.) with visual pass/fail indicators.
+**`repo-panopticon`** — Multi-repo health dashboard. Checks each repository against a suite
+of indicators in a compact 80-column table. Monorepo-aware: workspace packages are
+auto-discovered, grouped under their parent repo label, and checked individually.
 ```bash
-pkgck
+repo-panopticon <dirs...>                 # check one or more repos / globs
+repo-panopticon @plt/* @kud/* @srv/*      # typical full-org sweep
+repo-panopticon --no-pull @plt/*          # skip git pull/fetch
+repo-panopticon --limit 40 @srv/*         # cap displayed rows (default 80)
+repo-panopticon --concurrency 10 @srv/*   # increase parallelism (default 5)
 ```
+Columns: `mgr` (npm/yrn) · `pkg` (package.json version) · `git` (latest semver tag) ·
+`scop` (package scope) · `cir` (CircleCI — ✘ missing · L legacy · 1 orb v1 · 2 orb v2) ·
+`ESM` (`"type":"module"`) · `har` (.har hooks directory) · `lnt` (ESLint major version) ·
+`bbl` (Babel presence). Repo name shown in orange when dirty or sync failed.
 
 **`pkglint`** — Verifies required documentation files (README.md, LICENSE.md) are
 present before publishing.
@@ -252,8 +263,9 @@ count-lines-of-code [--src-only]
 
 ### CI/CD Integration
 
-**`circle-orb-it`** — Installs or updates CircleCI configuration (Leverege orb) and
-GitHub Dependabot settings for a repository. Preserves existing customizations.
+**`circle-orb-it`** — Installs or updates CircleCI configuration (Leverege orb v2.0.0) and
+GitHub Dependabot settings for a repository. Auto-detects npm vs yarn-berry and writes
+the appropriate template. Preserves existing customizations.
 ```bash
 circle-orb-it [--dry-run] [--debug]
 ```
@@ -340,6 +352,7 @@ BUILD_TOOLS_DEBUG=1 docker-to-registry
 
 | Tool | Replacement |
 |------|-------------|
+| `pkgck` | `repo-panopticon` |
 | `docker-to-registry-py` | `docker-to-registry` |
 | `chart-to-museum` | `push-my-chart` |
 | `refresh-py-idx` | No longer needed |

package/eslint.config.mjs

@@ -0,0 +1,13 @@
+import eslintConfig from '@leverege/eslint-config-leverege'
+
+export default [
+  ...eslintConfig,
+  {
+    rules: {
+      '@stylistic/curly-newline': 'off',
+      '@stylistic/key-spacing': 'off',
+      '@stylistic/object-curly-newline': 'off',
+      '@stylistic/nonblock-statement-body-position': 'off'
+    }
+  }
+]

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@leverege/build-tools",
-  "version": "2.102.0",
+  "version": "2.104.0",
   "description": "A collection of build / support tools for Leverege developers",
   "main": "index.js",
   "repository": {
@@ -9,14 +9,13 @@
   },
   "scripts": {
     "clean": "rm -rf lib",
-    "lint": "find ./src -name \\*.\\*js | xargs npx eslint --report-unused-disable-directives",
+    "lint": "eslint",
     "prepare": "[ $SKIP_PREPARE ] && exit 0 || ./src/hook-and-release.mjs",
     "test": "mocha -t 60000 --exit 'test/**/!(*e2e*).mjs'",
     "test-all": "mocha -t 60000 --exit 'test/**/**.mjs'",
     "prepack": "npm exec prepack"
   },
   "bin": {
-    "build-and-push": "src/build-and-push.sh",
     "build-cnpg-image": "src/build-cnpg-image.mjs",
     "build-tools": "src/build-tools.mjs",
     "change-up": "src/change-up.sh",
@@ -29,8 +28,6 @@
     "decrypt-secrets": "src/decrypt-secrets.sh",
     "dirty-git": "src/dirty-git.sh",
     "docker-to-registry": "src/docker-to-registry.mjs",
-    "docker-to-registry-py": "src/docker-to-registry-py.mjs",
-    "docker-to-registry.sh": "src/docker-to-registry.sh",
     "dockreate": "src/dockreate.sh",
     "encrypt-secrets": "src/encrypt-secrets.sh",
     "firebaseDeploy": "src/firebaseDeploy.mjs",
@@ -55,7 +52,7 @@
     "npm-link": "src/npm-link.sh",
     "npm-unlink": "src/npm-unlink.sh",
     "overwhelm": "src/overwhelm.mjs",
-    "pkgck": "src/pkgck.sh",
+    "pkgck": "echo '*** Use repo-panopticon instead! ***",
     "pkglint": "src/pkglint.sh",
     "prepack": "src/prepack.mjs",
     "prune-git": "src/prune-git.sh",
@@ -65,11 +62,10 @@
     "push-my-chart": "src/push-my-chart.mjs",
     "refresh-npm-token": "src/refresh-npm-token.mjs",
     "refresh-py-idx": "src/refresh-py-idx.sh",
+    "repo-panopticon": "src/repo-panopticon.mjs",
     "send-to-slack": "src/send-to-slack.mjs",
     "service-man": "src/service-man/service-man.mjs",
     "snapshot-cnpg": "src/snapshot-cnpg.mjs",
-    "tag-release": "src/tag-release.mjs",
-    "unleash": "src/unleash.mjs",
     "upload-model": "src/upload-model.sh",
     "yarn-build": "src/yarn-build.mjs"
   },
@@ -109,13 +105,14 @@
     "sloc": "^0.3.2",
     "superstruct": "^2.0.2",
     "tmp": "^0.2.5",
-    "toml": "^3.0.0",
+    "toml": "^4.1.1",
     "yaml": "^2.9.0",
     "zx": "^8.8.5"
   },
   "devDependencies": {
-    "@leverege/eslint-config-leverege": "^5.1.2",
+    "@leverege/eslint-config-leverege": "^9.0.1",
     "chai": "^6.2.2",
+    "eslint": "^9",
     "mocha": "^11.7.5",
     "npm": "^11.14.1"
   }

package/src/Docker.mjs

@@ -98,7 +98,7 @@ const generateDockerfile = async ( options ) => {
   // could probably be tightened up a little.
   try {
     fs.readdirSync( './docker' )
-  } catch ( err ) {
+  } catch ( err ) { // eslint-disable-line no-unused-vars
     warning( 'creating the missing ./docker directory' )
     fs.mkdirSync( './docker' ) // TODO: blindly assumes the mkdir succeeds
   }
@@ -120,7 +120,7 @@ const generateDockerfile = async ( options ) => {
   try {
     fs.writeFileSync( bashrcFile, bashrcTemplate )
     fs.appendFileSync( bashrcFile, bashrcPlugin )
-  } catch ( err ) {
+  } catch ( err ) { // eslint-disable-line no-unused-vars
     errorExit( `***ERROR: failed creation of the ${bashrcFile} file` )
   }
 
@@ -136,7 +136,7 @@ const generateDockerfile = async ( options ) => {
     if ( build.length === 0 ) {
       errorExit( `***ERROR: the build dir is empty - ${chalk.yellow( 'npm run build' )}` )
     }
-  } catch ( err ) {
+  } catch ( err ) { // eslint-disable-line no-unused-vars
     errorExit( `***ERROR: expected the ${buildDir} directory to exist - ${chalk.yellow( 'npm run build' )}` )
   }
 

package/src/Utils.mjs

@@ -132,7 +132,7 @@ export const shellCmd = async ( cmd, opts = {} ) => {
     } else {
       const parsed = parse( cmd ).filter( x => typeof x === 'string' )
       // the leading ; forces a break from previous line otherwise trailing ) and leading [ collide
-      ;[ command, ...args ] = parsed // eslint-disable-line semi-style
+      ;[ command, ...args ] = parsed // eslint-disable-line @stylistic/semi-style
     }
 
     debug( { command, args }, '<== execa $ invoke' )
@@ -271,7 +271,7 @@ export const getGitBranchAndUpstream = async () => {
   let upstream = 'no-upstream'
   try {
     upstream = await tryCommand( `git rev-parse --abbrev-ref ${branch}@{u}`, 'getGitUpstream' ) ?? 'no-upstream'
-  } catch ( error ) {}
+  } catch ( error ) {} // eslint-disable-line no-unused-vars
   return { branch, upstream }
 }
 
@@ -534,6 +534,7 @@ export const parseHelmChart = async ( helmroot = './helm' ) => {
   const chartName = chartYaml.name
   const chartLibrary = chartYaml?.type === 'library'
   const chartVersion = chartYaml.version
+  const chartAppVersion = chartYaml.appVersion
   const chartPackage = `${chartName}-${chartVersion}.tgz`
   const valuesText = fs.readFileSync( `${helmroot}/values.yaml`, 'utf8' )
   const valuesYaml = YAML.load( valuesText )
@@ -573,6 +574,7 @@ export const parseHelmChart = async ( helmroot = './helm' ) => {
     chartLibrary,
     chartPackage,
     chartVersion,
+    chartAppVersion,
     hasDependencies,
     depsOutOfSync,
     imageRegistry,
@@ -708,12 +710,12 @@ export const analyzeRepository = async ( giveGuidance ) => {
   let wsName
   try {
     wsName = await shellCmd( 'npm exec -c pwd -ws' )
-  } catch ( err ) { }
+  } catch ( err ) { } // eslint-disable-line no-unused-vars
 
   let gitRemote
   try {
     gitRemote = await shellCmd( 'git config --get remote.origin.url' )
-  } catch ( err ) { }
+  } catch ( err ) { } // eslint-disable-line no-unused-vars
 
   // TODO: Add in Define VERBOSE_DOCKREATE_NPM="yes" for verbose build logging
   // TODO: Validity check for version and package.json

package/src/build-cnpg-image.mjs

@@ -139,7 +139,7 @@ const buildAndPushImage = async () => {
   const dockerfilePath = path.join( buildDir, 'Dockerfile' )
 
   try {
-    fs.writeFileSync( dockerfilePath, renderedDockerfile )
+    fs.writeFileSync( dockerfilePath, renderedDockerfile ) // eslint-disable-line security/detect-non-literal-fs-filename
 
     log( chalk.yellow( '\nBuilding Docker image...' ) )
     const cloudBuildSubmit = 'gcloud builds submit --project leverege-registry'

package/src/chart-compass.mjs

@@ -96,7 +96,7 @@ if ( args.init ) {
     errorExit( '\n***ERROR: expected to run in the root of a platform-k8s branch - no overwhelm.yaml present' )
   }
 
-  if ( fs.existsSync( args.compass ) ) {
+  if ( fs.existsSync( args.compass ) ) { // eslint-disable-line security/detect-non-literal-fs-filename
     errorExit( `\n***ERROR: ${args.compass} compass file already exists - not overwriting` )
   }
 
@@ -107,12 +107,12 @@ if ( args.init ) {
 
 // simply wraps the YAML read/sync
 const buildCompassMap = ( yamlFile ) => {
-  if ( !fs.existsSync( yamlFile ) ) {
+  if ( !fs.existsSync( yamlFile ) ) { // eslint-disable-line security/detect-non-literal-fs-filename
     debug( `chart-compass file ${yamlFile} does not exist` )
     process.exit( 1 )
   }
 
-  const chartCompass = YAML.load( fs.readFileSync( yamlFile, 'utf8' ) )
+  const chartCompass = YAML.load( fs.readFileSync( yamlFile, 'utf8' ) ) // eslint-disable-line security/detect-non-literal-fs-filename
   return chartCompass
 }
 

package/src/chart-to-registry.mjs

@@ -14,9 +14,10 @@ import {
   errorExit,
   establishOauthAccess,
   log,
-  warning,
   parseHelmChart,
+  proceed,
   shellCmd,
+  warning,
 } from './Utils.mjs'
 
 const commandLineOptions = [ // Use commandLineOptions to tie into the Usage statements
@@ -46,6 +47,11 @@ const commandLineOptions = [ // Use commandLineOptions to tie into the Usage sta
     type : Boolean,
     description : '{green update artifact registry auth token and exit}',
   },
+  {
+    name : 'ignore-appv',
+    type : Boolean,
+    description : '{green ignore the Char.yaml appVersion format}',
+  },
   {
     name : 'dry-run',
     type : Boolean,
@@ -98,6 +104,11 @@ if ( semverLt( process.version, minNodejsVersion ) ) {
 // load up all we need to know about the chart
 const helmChart = await parseHelmChart( args.chart )
 
+if ( !helmChart.chartAppVersion.startsWith( 'v' ) && !args.ignoreAppv ) {
+  warning( 'Chart.yaml appVersion does not begin with the letter "v"\n' )
+  await proceed( 'You sure about this?' )
+}
+
 // helm push helm-package.tgz oci://REGISTRY_LOCATION/REGISTRY_PROJECT/REGISTRY_REPOSITORY/charts
 //
 const registryLocation = args.location || helmChart.registryLocation
@@ -140,7 +151,7 @@ try {
   } else {
     errorExit( `\n***ERROR: chart ${chartName} version ${chartVersion} already exists` )
   }
-} catch ( error ) {
+} catch ( error ) { // eslint-disable-line no-unused-vars
   debug( `Chart ${chartName} version ${chartVersion} does not exist - proceeding` )
 }
 

package/src/circle-orb-it/{circle-orb.yml → circle-npm.yml}

@@ -1,25 +1,28 @@
 version: 2.1
 
 orbs:
-  leverege: leverege/circle-nodejs-ci@1.2.6
+  leverege: leverege/circle-nodejs-ci@2.0.0
 
 workflows:
   build-and-verify:
     jobs:
       - leverege/install-dependencies:
           context: npm
-          # pkg-manager: yarn-berry   # uncomment for Yarn Berry repos
       - leverege/code-analysis:
+          context: npm
           requires:
             - leverege/install-dependencies
       - leverege/code-coverage:
+          context:
+            - npm
+            - leverege-gcp                # provides GCP_SA_KEY_JSON for tests that need GCP access
           coverage-threshold: 70          # percentage threshold for functions, lines, and statements
-          context: leverege-gcp           # provides GCP_SA_KEY_JSON for tests that need GCP access
           enable-redis: false             # set to true if tests require a live Redis connection
           cache-config: '{"type":"inMemory"}' # sets CACHE_CONFIG env var - options: inMemory, redis
           requires:
             - leverege/install-dependencies
       - leverege/generate-docs:
+          context: npm
           requires:
             - leverege/install-dependencies
       - leverege/mirror-to-github:

package/src/circle-orb-it/circle-orb-it.mjs

@@ -11,14 +11,19 @@ import {
   enableDebug,
   errorExit,
   getGitRootDirectory,
+  getIsYarnProject,
   log,
+  mkdirSafe,
   proceed,
+  warning,
 } from '../Utils.mjs'
 
-const ORB_VERSION = '1.2.6' // Keep this in sync with the version in circle-orb.yml
+const ORB_VERSION = '2.0.0' // Keep this in sync with the versions in circle-npm.yml and circle-yarn.yml
 const CIRCLE_DIR = '.circleci'
 const CIRCLE_YML = `${CIRCLE_DIR}/config.yml`
-const CIRCLE_YML_TEMPLATE = path.join( BUILD_TOOLS_ROOT, 'circle-orb-it', 'circle-orb.yml' )
+const CIRCLE_NPM_TEMPLATE = path.join( BUILD_TOOLS_ROOT, 'circle-orb-it', 'circle-npm.yml' )
+const CIRCLE_YARN_TEMPLATE = path.join( BUILD_TOOLS_ROOT, 'circle-orb-it', 'circle-yarn.yml' )
+const CIRCLE_YML_TEMPLATE = isYarn => ( isYarn ? CIRCLE_YARN_TEMPLATE : CIRCLE_NPM_TEMPLATE )
 const GITHUB_DIR = '.github'
 const DEPENDABOT_FILE = `${GITHUB_DIR}/dependabot.yml`
 const DEPENDABOT_TEMPLATE = path.join( BUILD_TOOLS_ROOT, 'circle-orb-it', 'dependabot.yml' )
@@ -52,9 +57,13 @@ if ( !gitRoot ) {
   errorExit( 'This command must be run inside a git repository' )
 }
 
+const isYarn = getIsYarnProject( gitRoot )
+const circleTemplate = CIRCLE_YML_TEMPLATE( isYarn )
+log( chalk.blue( `\nPackage manager: ${isYarn ? 'yarn-berry' : 'npm'}` ) )
+
 // Warn if this looks like a Python project since the orb is Node.js only
 if ( fs.existsSync( path.join( gitRoot, 'pyproject.toml' ) ) ) {
-  log( chalk.yellow( '\n***WARNING: pyproject.toml detected - the leverege orb is Node.js only' ) )
+  warning( 'pyproject.toml detected - the leverege orb is Node.js only' )
   await proceed( 'Continue anyway?' )
 }
 
@@ -119,7 +128,7 @@ if ( !writeCircleYml && !writeDependabot ) {
 
 if ( options.dryRun ) {
   if ( writeCircleYml ) {
-    let dryRunContent = fs.readFileSync( CIRCLE_YML_TEMPLATE, 'utf-8' )
+    let dryRunContent = fs.readFileSync( circleTemplate, 'utf-8' )
     if ( redisEnabled ) {
       dryRunContent = dryRunContent
         .replace( 'enable-redis: false', 'enable-redis: true' )
@@ -142,10 +151,8 @@ if ( options.dryRun ) {
 // ---------- Write ----------
 
 if ( writeCircleYml ) {
-  if ( !fs.existsSync( circleDir ) ) {
-    fs.mkdirSync( circleDir, { recursive : true } )
-  }
-  let circleYmlContent = fs.readFileSync( CIRCLE_YML_TEMPLATE, 'utf-8' )
+  mkdirSafe( circleDir )
+  let circleYmlContent = fs.readFileSync( circleTemplate, 'utf-8' )
   if ( redisEnabled ) {
     circleYmlContent = circleYmlContent
       .replace( 'enable-redis: false', 'enable-redis: true' )
@@ -162,9 +169,7 @@ if ( writeCircleYml ) {
 }
 
 if ( writeDependabot ) {
-  if ( !fs.existsSync( githubDir ) ) {
-    fs.mkdirSync( githubDir, { recursive : true } )
-  }
+  mkdirSafe( githubDir )
   fs.writeFileSync( dependabotPath, fs.readFileSync( DEPENDABOT_TEMPLATE, 'utf-8' ) )
   log( chalk.green( `Dependabot config written => ${dependabotPath}` ) )
 }

package/src/circle-orb-it/circle-yarn.yml

@@ -0,0 +1,25 @@
+version: 2.1
+
+orbs:
+  leverege: leverege/circle-nodejs-ci@2.0.0
+
+workflows:
+  build-and-verify:
+    jobs:
+      - leverege/code-analysis:
+          context: npm
+          pkg-manager: yarn-berry
+      - leverege/code-coverage:
+          context:
+            - npm
+            - leverege-gcp                # provides GCP_SA_KEY_JSON for tests that need GCP access
+          pkg-manager: yarn-berry
+          coverage-threshold: 70          # percentage threshold for functions, lines, and statements
+          enable-redis: false             # set to true if tests require a live Redis connection
+          cache-config: '{"type":"inMemory"}' # sets CACHE_CONFIG env var - options: inMemory, redis
+      - leverege/generate-docs:
+          context: npm
+          pkg-manager: yarn-berry
+      - leverege/mirror-to-github:
+          context: github-mirror
+          # github-team: platform   # available teams: platform, utilities, pitcrew, recovr

package/src/custom-images/bitnami-audits/bitnami-audit.mjs

@@ -123,7 +123,7 @@ const runner = async ( projects ) => {
     log( `Project [${project}]` )
     try {
       await prepareContext( project ) // eslint-disable-line no-await-in-loop
-    } catch ( error ) {
+    } catch ( error ) { // eslint-disable-line no-unused-vars
       log( `Project or context is invalid, skipping [${chalk.redBright( project )}]` )
       continue
     }

package/src/dashboard/DashboardManager.mjs

@@ -59,6 +59,7 @@ export default class DashboardManager {
       // leaf directory — apply JSON files using basename as the Grafana folder name
       const folderName = path.basename( dir )
       for ( const file of jsonFiles ) {
+        // eslint-disable-next-line no-await-in-loop
         const name = await this.#applyDashboard( folderName, path.join( dir, file.name ) )
         if ( name ) applied.add( name )
       }
@@ -79,6 +80,7 @@ export default class DashboardManager {
           .filter( f => f.endsWith( '.json' ) )
 
         for ( const file of files ) {
+          // eslint-disable-next-line no-await-in-loop
           const name = await this.#applyDashboard( folder.name, path.join( dir, folder.name, file ) )
           if ( name ) applied.add( name )
         }
@@ -159,6 +161,7 @@ export default class DashboardManager {
     if ( subDirs.length === 0 ) {
       const folderName = path.basename( dir )
       for ( const file of jsonFiles ) {
+        // eslint-disable-next-line no-await-in-loop
         const name = await this.#applyDashboard( folderName, path.join( dir, file.name ), local )
         if ( name ) applied.add( name )
       }
@@ -168,6 +171,7 @@ export default class DashboardManager {
           .filter( f => f.endsWith( '.json' ) )
 
         for ( const file of files ) {
+        // eslint-disable-next-line no-await-in-loop
           const name = await this.#applyDashboard( folder.name, path.join( dir, folder.name, file ), local )
           if ( name ) applied.add( name )
         }
@@ -259,6 +263,7 @@ export default class DashboardManager {
         continue
       }
       try {
+        // eslint-disable-next-line no-await-in-loop
         await api.deleteNamespacedConfigMap( { name, namespace : this.#namespace } )
         log( chalk.red( `  ✗ pruned ${name}` ) )
       } catch ( e ) {
@@ -290,6 +295,7 @@ export default class DashboardManager {
         continue
       }
       try {
+        // eslint-disable-next-line no-await-in-loop
         await api.deleteNamespacedConfigMap( { name, namespace : this.#namespace } )
         log( chalk.red( `  ✗ pruned (local) ${name}` ) )
       } catch ( e ) {

package/src/firebaseDeploy.mjs

@@ -79,7 +79,6 @@ const targets = Object
           // Store the longest length of site id for formatting later on
           maxSiteIdLength = Math.max( siteId.length, maxSiteIdLength )
 
-          // eslint-disable-next-line no-param-reassign
           prev[alias] = {
             projectId,
             siteId,

package/src/firebaseServe.mjs

@@ -65,7 +65,6 @@ const targets = Object
       .entries( config.hosting )
       .forEach( ( [ siteId, aliases ] ) => {
         aliases.forEach( ( alias ) => {
-          // eslint-disable-next-line no-param-reassign
           prev[alias] = {
             projectId,
             siteId,

package/src/overwhelm.mjs

@@ -26,7 +26,7 @@ import {
 } from './Utils.mjs'
 
 const optionsDefinitions = [
-  /* eslint-disable no-multi-spaces */
+  /* DISABLED eslint-disable no-multi-spaces */
   { name : 'context',    alias : 'x', type : String, default : '' },
   { name : 'deprecated', alias : 'd', type : Boolean, default : false },
   { name : 'section',    alias : 's', type : String, default : 'default' },
@@ -38,7 +38,7 @@ const optionsDefinitions = [
   { name : 'genvals',    alias : 'g', type : Boolean, default : false }, // only generates values.yaml
   { name : 'matchk8s',   alias : 'm', type : Boolean, default : false }, // checks k8s context
   { name : 'verbose',    alias : 'v', type : Boolean, default : false },
-  /* eslint-enable no-multi-spaces */
+  /* DISABLED eslint-enable no-multi-spaces */
 ]
 const args = cliArgs( optionsDefinitions )
 
@@ -307,12 +307,12 @@ ahoy.forEach( ( dir ) => {
     const valuesAll = YAML.load( valuesOut ) || {}
     const valuesSvc = {
       service : {
-        /* eslint-disable no-template-curly-in-string */
+        /* DISABLE eslint-disable no-template-curly-in-string */
         project_id : '${PROJECT_ID}',
         project_name : '${PROJECT_NAME}',
         host : '${HOST}',
         ingress : '${INGRESS}',
-        /* eslint-enable no-template-curly-in-string */
+        /* DISABLE eslint-enable no-template-curly-in-string */
       }
     }
     const valuesMerged = YAML.dump( deepmerge( valuesSvc, valuesAll ), { quotingType : '"' } )
@@ -342,33 +342,11 @@ if ( over.PROJECT_NAME === 'minikube' ) {
 const projectId = over.PROJECT_ID
 const zone = over.GCE_REGION + ( over.GCE_ZONE ? `-${over.GCE_ZONE}` : '' )
 
-if ( !clusterName ) {
-  /* eslint-disable quotes */
-  log( chalk.yellow(
-    `
-
-    The overwhelm.yaml file in this branch is missing a CLUSTER_NAME setting,
-    which was added to support cluster names that do not match the project
-    name. Since the mismatch is rare, you can probably get by with simply
-    adding this line following PROJECT_NAME in the default section of
-    overwhelm.yaml:
-
-      ${chalk.bold.green( `CLUSTER_NAME   : '\${PROJECT_NAME}'` )}
-
-    However, if the CLUSTER_NAME is not the same as the PROJECT_NAME then go
-    ahead and add the CLUSTER_NAME setting to the ${chalk.bold.green( projectId )} section of the
-    overwhelm.yaml file.
-
-    ` ) )
-  /* eslint-enable quotes */
-  process.exit( 1 )
-}
-
 // the kubectl call will throw if there isn't an existing context
 let currentCtx = '_no_current_context_'
 try {
   currentCtx = await shellCmd( 'kubectl config current-context' )
-} catch ( err ) {
+} catch ( err ) { // eslint-disable-line no-unused-vars
   // err && log( `Caught Error =>[${err}]` )
 }
 
@@ -386,7 +364,7 @@ if ( args.fetchctx ) {
   if ( !matchedCtx ) {
     try {
       await shellCmd( cmd )
-    } catch ( err ) {
+    } catch ( err ) { // eslint-disable-line no-unused-vars
       process.exit( 61 ) // ECONNREFUSED
     }
   }

package/src/push-my-chart.mjs

@@ -9,7 +9,9 @@ import {
   errorExit,
   log,
   parseHelmChart,
-  shellCmd
+  proceed,
+  shellCmd,
+  warning,
 } from './Utils.mjs'
 
 const commandLineOptions = [
@@ -56,11 +58,18 @@ if ( args.help ) {
 const {
   chartName,
   chartVersion,
+  chartAppVersion,
   depsOutOfSync,
 } = await parseHelmChart( args.chart )
 
-log( chalk.green( `Chart   => ${chalk.bold.yellow( chartName )}` ) )
-log( chalk.green( `Version => ${chalk.bold.yellow( chartVersion )}` ) )
+log( chalk.green( `Chart      => ${chalk.bold.yellow( chartName )}` ) )
+log( chalk.green( `Version    => ${chalk.bold.yellow( chartVersion )}` ) )
+log( chalk.green( `AppVersion => ${chalk.bold.yellow( chartAppVersion )}` ) )
+
+if ( !chartAppVersion.startsWith( 'v' ) ) {
+  warning( 'Chart.yaml appVersion does not begin with the letter "v"\n' )
+  await proceed( 'You sure about this?' )
+}
 
 if ( depsOutOfSync ) {
   try {
@@ -89,7 +98,7 @@ if ( args.noRegistry ) {
 
 try {
   log( chalk.green( '  ... pushing to the Artifact Registry' ) )
-  const registry = await shellCmd( `chart-to-registry --chart ${args.chart}` )
+  const registry = await shellCmd( `chart-to-registry --chart ${args.chart} --ignore-appv` )
   debug( { registry }, '<==chart-to-registry' )
   process.exit( museumStatus )
 } catch ( ex ) {