wp-advads 1.0.33 → 1.0.34

package/.github/workflows/test.yml

@@ -0,0 +1,17 @@
+name: test
+
+on:
+  push:
+  pull_request:
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 22
+          cache: npm
+      - run: npm ci
+      - run: npm test

package/CHANGELOG.md

@@ -4,9 +4,31 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/), and this project adheres to [Semantic Versioning](https://semver.org/).
 
 ---
+## [1.0.34] - 2026-09-11
+
+### Changed
+- `init` only writes `wp.advads`; this CLI is commands inside an existing plugin, not a full-plugin generator.
+- `build` runs the consumer `package.json` `scripts.build` when present, and no longer mutates release commit messages.
+- Release git commit uses an explicit file list and runs hooks (`--no-verify` and `git add .` are gone).
+- Shell-outs fail closed; large builds stream via `spawn`.
+- Scaffold writes stay inside the project root and refuse to overwrite unless `--force`.
+- Nested `js` / `css` / `view` paths accept `/` and `\`.
+- POT headers come from `author` in config; `pot` docs match that it writes a `.pot`.
+- Requires Node `>=18`. Package description grammar and license (ISC) match the repo.
+
+### Fixed
+- `view` and `updates` compile with the real `init` config shape (`description`, `misc.prefix`).
+- Class `--heading` plus `-i`/`-g`/`-r`/`-s` selects the right template.
+- SemVer pre-release increment (`alpha.2` → `alpha.3`) and invalid tags.
+- `getLatestTag`, GitHub permission/fetch, and `updateFileContent` stop release on error instead of continuing.
+- Singleton stub constructor is private; GlotPress locale ids are sanitized.
+
+### Removed
+- Laravel Mix / PHPCS / plugin boilerplate templates, `fs-extra`, and the `async` package.
+
 ## [1.0.28] - 2025-04-20
 ### Added
-- `pot` command to generate `.po` and `.mo` translation files.
+- `pot` command to generate `.pot` translation files.
 - `translations` command to download translation files from GlotPress.
 
 ## [1.0.23] - 2025-04-15

package/LICENSE

@@ -0,0 +1,15 @@
+ISC License
+
+Copyright (c) 2026 Shakeeb Ahmed
+
+Permission to use, copy, modify, and/or distribute this software for any
+purpose with or without fee is hereby granted, provided that the above
+copyright notice and this permission notice appear in all copies.
+
+THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
+MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
+WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
+ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
+OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.

package/README.md

@@ -4,9 +4,7 @@
 
 
 [![npm version](https://img.shields.io/npm/v/wp-advads.svg)](https://www.npmjs.com/package/wp-advads)
-[![JavaScript Style Guide](https://img.shields.io/badge/code_style-standard-brightgreen.svg)](https://standardjs.com)
-[![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](LICENSE)
-![Version](https://img.shields.io/badge/version-2.0.0-yellow.svg)
+[![License: ISC](https://img.shields.io/badge/License-ISC-blue.svg)](LICENSE)
 
 <p align="center">
 	<img src="https://img.icons8.com/nolan/256/wordpress.png"/>
@@ -37,6 +35,7 @@ Initializes a new project configuration file.
 npx wp-advads init
 ```
 
+`wp.advads` lives in the nearest ancestor directory that already has that file. `init` creates it in the current directory.
 
 #### `release`
 Creates a new release for the plugin.
@@ -48,15 +47,15 @@ npx wp-advads release
 #### `class <classname> [options]`
 Creates a new PHP class file with optional templates.
 ```bash
-npx wp-advads class MyClass -h "Class Heading" -d "Description" -s
+npx wp-advads class MyClass --heading "Class Heading" --description "Description" -s
 ```
 
 ##### Required Argument:
 - `<classname>` – The name of the class.
 
 ##### Optional Argument:
-- `-h <heading>` – File heading
-- `-d <description>` – File description
+- `--heading <heading>` – File heading
+- `--description <description>` – File description
 - `-i` – Initializer interface
 - `-g` – Integration interface
 - `-r` – REST interface
@@ -113,7 +112,7 @@ npx wp-advads css styles -h "Main CSS" -d "Plugin styles"
 - `-d <description>` – Description comment for the file.
 
 #### `pot`
-Generates `.po` and `.mo` translation files.
+Generates a `.pot` translation template.
 ```bash
 npx wp-advads pot
 ```
@@ -129,6 +128,8 @@ A config file is required for all commands **except** init.
 
 If one is missing, you'll be prompted to create it with `wp-advads init`.
 
+`class`, `view`, `updates`, `js`, and `css` will not overwrite an existing file unless you pass `--force`.
+
 ### 👤 Author
 Shakeeb Ahmed
 GitHub: [@meshakeeb](http://github.com/meshakeeb)

package/package.json

@@ -1,8 +1,8 @@
 {
   "type": "module",
   "name": "wp-advads",
-  "version": "1.0.33",
-  "description": "Create a Advanced Ads wordPress plugin eco system.",
+  "version": "1.0.34",
+  "description": "Create an Advanced Ads WordPress plugin ecosystem.",
   "author": {
     "name": "Shakeeb Ahmed",
     "email": "me@shakeebahmed.com",
@@ -13,7 +13,8 @@
     "wp-advads": "src/app.js"
   },
   "scripts": {
-    "start": "node src/app.js"
+    "start": "node src/app.js",
+    "test": "node --experimental-test-module-mocks --test"
   },
   "repository": {
     "type": "git",
@@ -25,15 +26,16 @@
     "generator"
   ],
   "license": "ISC",
+  "engines": {
+    "node": ">=18"
+  },
   "bugs": {
-    "url": "https://github.com/advanced-ads/npm-scaffolding/issues"
+    "url": "https://github.com/advanced-ads/wp-advads/issues"
   },
-  "homepage": "https://github.com/advanced-ads/npm-scaffolding#readme",
+  "homepage": "https://github.com/advanced-ads/wp-advads#readme",
   "dependencies": {
-    "async": "^3.2.6",
     "chalk": "^5.4.1",
     "commander": "^13.1.0",
-    "fs-extra": "^11.3.0",
     "inquirer": "^12.5.2",
     "lodash": "^4.17.21",
     "log-symbols": "^7.0.0"

package/src/app.js

@@ -5,6 +5,9 @@
  */
 import chalk from 'chalk'
 import { program } from 'commander'
+import { readFileSync } from 'node:fs'
+import { dirname, join } from 'node:path'
+import { fileURLToPath } from 'node:url'
 
 /**
  * Commands
@@ -12,6 +15,10 @@ import { program } from 'commander'
 import * as commands from './commands/index.js'
 import { configFileExists, onNewLine } from './utilities/index.js'
 
+const { version } = JSON.parse(
+	readFileSync( join( dirname( fileURLToPath( import.meta.url ) ), '../package.json' ), 'utf8' )
+)
+
 /**
  * App
  */
@@ -19,7 +26,7 @@ const app = async () => {
     console.log(
         [
             chalk.hex('#FADC00').inverse.bold('Advanced Ads WordPress Plugin Toolkit'),
-            chalk.white( 'v2.0.0' ),
+            chalk.white( 'v' + version ),
             chalk.dim( 'by Shakeeb Ahmed' )
         ].join(" ")
     );
@@ -31,7 +38,7 @@ const app = async () => {
 	program
 		.name('wp-advads')
 		.description('CLI to Advanced Ads scaffolding utilities')
-		.version('2.0.0')
+		.version(version)
 		.hook('preAction', (thisCommand, actionCommand) => {
 			if ( 'init' !== actionCommand.name() && ! configFileExists() ) {
 				onNewLine(chalk.bgRed.white('Config file not found. Run `wp-advads init` to create a new config file.'));
@@ -52,12 +59,6 @@ const app = async () => {
 		.description('Build the plugin/theme according to the config file')
 		.action(commands.build);
 
-	// Command: plugin
-	// program
-	// 	.command('plugin')
-	// 	.description('Create a new plugin')
-	// 	.action(commands.plugin);
-
 	// Command: release
 	program
 		.command('release')
@@ -69,12 +70,13 @@ const app = async () => {
 		.command('class')
 		.description('Create a new class file')
 		.argument('<classname>', 'Classname')
-		.option('-h <heading>', 'Heading of the file')
-		.option('-d <description>', 'Description of the file')
+		.option('--heading <heading>', 'Heading of the file')
+		.option('--description <description>', 'Description of the file')
 		.option('-i', 'Initializer interface template')
 		.option('-g', 'Integration interface template')
 		.option('-r', 'Rest interface template')
 		.option('-s', 'Singleton template')
+		.option('--force', 'Overwrite an existing file')
 		.action(commands.classFile);
 
 	// Command: view
@@ -83,6 +85,7 @@ const app = async () => {
 		.description('Create a new view')
 		.argument('<viewname>', 'name of the view')
 		.argument('[heading]', 'Heading of the file')
+		.option('--force', 'Overwrite an existing file')
 		.action(commands.views);
 
 	// Command: updates
@@ -90,6 +93,7 @@ const app = async () => {
 		.command('updates')
 		.description('Updates for the plugin')
 		.argument('<version>', 'version number')
+		.option('--force', 'Overwrite an existing file')
 		.action(commands.updates);
 
 	// Command: js
@@ -99,6 +103,7 @@ const app = async () => {
 		.argument('<filename>', 'name of the file')
 		.option('-h <heading>', 'Heading of the file')
 		.option('-d <description>', 'Description of the file')
+		.option('--force', 'Overwrite an existing file')
 		.action(commands.javascript);
 
 	// Command: css
@@ -108,12 +113,13 @@ const app = async () => {
 		.argument('<filename>', 'name of the file')
 		.option('-h <heading>', 'Heading of the file')
 		.option('-d <description>', 'Description of the file')
+		.option('--force', 'Overwrite an existing file')
 		.action(commands.css);
 
 	// Command: wp-pot
 	program
 		.command('pot')
-		.description('Create po and mo files')
+		.description('Create a .pot file')
 		.action(commands.wpPot);
 
 	// Command: wp-glotpress

package/src/commands/build.js

@@ -3,80 +3,93 @@
  */
 import fs from 'fs';
 import chalk from 'chalk'
-import { waterfall } from 'async'
 import logSymbols from 'log-symbols'
 
 /**
  * Node dependencies
  */
-import { pluginData } from './release.js';
 import wpPot from './wp-pot.js';
 import { heading, onSameLine, execCommand } from '../utilities/index.js'
 
-export default function wpBuild(nextProcess = null) {
+export default function wpBuild(nextProcess = null, commitMessages = null) {
 	heading('Building the plugin...')
 	console.log('------------------------------------------------');
 
-	waterfall(
-        [
-            (next) => {
-				process.stdout.write('Clearing packages folder...')
-				try {
-					fs.rmSync( 'packages', { recursive: true, force: true } );
-					onSameLine(`${logSymbols.success} Cleared packages folder`)
-				} catch ( _ ) {}
-				next();
-			},
-            (next) => {
-				process.stdout.write('Installing PHP dependencies...')
-
-				execCommand(
-					'composer install -o -a -n --no-dev --no-scripts',
-					function() {
-						onSameLine(`${logSymbols.success} Installed PHP dependencies`);
-						process.stderr.write('Composer dumping....')
-
-						execCommand('composer dump', function() {
-							onSameLine(`${logSymbols.success} Composer updated`)
-							pluginData.commitMessages.push('update 3rd party dependecies')
-							next();
-						})
-					},
-					{
-						env: { ...process.env, COMPOSER_VENDOR_DIR: 'packages' },
-					}
-				);
-
-			},
-            (next) => {
-				process.stdout.write('Building plugin resources...')
-				execCommand( 'npx wp-scripts build', function() {
-					onSameLine(`${logSymbols.success} Built plugin resources`);
-					pluginData.commitMessages.push('Build plugin resources')
-					next();
-				});
-			},
-            (next) => {
-				console.log('')
-				wpPot(next);
-			},
-        ],
-        ( err, results ) => {
-            if (err) {
-                console.log('');
-                console.log( `${logSymbols.error} ${chalk.bold.red(`We failed somewhere! Work hard mate...`)}` )
-				if (typeof nextProcess === 'function') {
-					nextProcess(true);
-				}
-                return
-            }
-
+	run( commitMessages )
+		.then( () => {
 			if (typeof nextProcess === 'function') {
 				nextProcess();
 			} else {
 				console.log('');
 				console.log( `${logSymbols.success} ${chalk.bold.green(`All done!`)}` )
 			}
-        }
-    );
+		} )
+		.catch( ( err ) => {
+			console.log('');
+			console.log( `${logSymbols.error} ${chalk.bold.red(`We failed somewhere! Work hard mate...`)}` )
+			if (typeof nextProcess === 'function') {
+				nextProcess(true);
+			}
+		} )
+}
+
+function run( commitMessages ) {
+	return ( async () => {
+		process.stdout.write('Clearing packages folder...')
+		try {
+			fs.rmSync( 'packages', { recursive: true, force: true } );
+			onSameLine(`${logSymbols.success} Cleared packages folder`)
+		} catch ( err ) {
+			onSameLine(`${logSymbols.error} ${err.message}`)
+			throw err
+		}
+
+		process.stdout.write('Installing PHP dependencies...')
+
+		const composerEnv = {
+			env: { ...process.env, COMPOSER_VENDOR_DIR: 'packages' },
+		}
+
+		await runExec( 'composer install -o -a -n --no-dev --no-scripts', composerEnv )
+		onSameLine(`${logSymbols.success} Installed PHP dependencies`);
+		process.stderr.write('Composer dumping....')
+
+		await runExec( 'composer dump', composerEnv )
+		onSameLine(`${logSymbols.success} Composer updated`)
+		commitMessages?.push( 'update 3rd party dependencies' )
+
+		process.stdout.write('Building plugin resources...')
+		let buildScript
+		try {
+			buildScript = JSON.parse( fs.readFileSync( 'package.json', 'utf8' ) ).scripts?.build
+		} catch {
+			buildScript = null
+		}
+
+		if ( ! buildScript ) {
+			onSameLine(`${logSymbols.warning} No package.json build script, skipping`)
+		} else {
+			await runExec( 'npm run build' )
+			onSameLine(`${logSymbols.success} Built plugin resources`);
+			commitMessages?.push( 'Build plugin resources' )
+		}
+
+		console.log('')
+		await new Promise( ( resolve, reject ) => {
+			wpPot( ( err ) => err ? reject( err ) : resolve() )
+		} )
+	} )()
+}
+
+function runExec( cmd, options ) {
+	return new Promise( ( resolve, reject ) => {
+		execCommand( cmd, ( stdout, error, stderr ) => {
+			if ( error ) {
+				onSameLine(`${logSymbols.error} ${stderr || error.message}`)
+				reject( error )
+				return
+			}
+			resolve( stdout )
+		}, options )
+	} )
 }

package/src/commands/class.js

@@ -1,8 +1,3 @@
-/**
- * External Dependencies
- */
-import capitalize from 'lodash/capitalize.js';
-
 /**
  * Node Dependencies
  */
@@ -11,53 +6,52 @@ import path from 'path';
 /**
  * Internal Dependencies
  */
-import { getSetting, writeFile, filenameToHeading, heading, getProjectRoot, msgErrorTitle, getTemplateFile, compileTemplate, runCommand } from "../utilities/index.js";
+import { getSetting, filenameToHeading, heading, getProjectRoot, writeFromTemplate } from "../utilities/index.js";
 
 function templateName(options)  {
-	const length = Object.keys(options).length
+	if ( options.i ) {
+		return 'class-initializer';
+	}
 
-	if ( 0 === length ) {
-		return 'class-empty';
+	if ( options.g ) {
+		return 'class-integration';
 	}
 
-	const hash = {
-		'i': 'class-initializer',
-		'g': 'class-integration',
-		'r': 'class-rest',
-		's': 'class-singleton',
+	if ( options.r ) {
+		return 'class-rest';
 	}
-	const has = Object.keys(options)[0]
 
-	return hash[has] ?? 'class-empty';
+	if ( options.s ) {
+		return 'class-singleton';
+	}
+
+	return 'class-empty';
 }
 
 export default (classname, options) => {
 	heading('Creating class file...')
     const {
-		h: header = null,
-		d: description = null,
+		heading: header = null,
+		description = null,
 	} = options
 
-	try {
-		const template = templateName(options);
-		const namespace = classname.split('\\');
-		const paths = classname.toLowerCase().split('\\');
-		const filename = paths.pop().replace(/_/g, '-') + '.php';
-		const folder = path.join(getProjectRoot(), getSetting('paths.php'), paths.join('/'));;
-
-		// Data
-		const data = getSetting();
-		data.heading = header || filenameToHeading(filename) + ' template file';
-		data.description = description || 'Brief description of the styles in this file';
-		data.className = namespace.pop();
-		data.namespace = namespace.length > 0 ? '\\' + namespace.join('\\') : '';
-
-		const content = compileTemplate(getTemplateFile(`files/${template}.php`), data);
-		writeFile(folder, `class-${filename}`, content);
-		runCommand( 'composer', [ 'dump' ], () => {} );
-	}
-	catch (err) {
-		msgErrorTitle('We failed!!!');
-		throw err;
-	}
+	const template = templateName(options);
+	const namespace = classname.split('\\');
+	const paths = classname.toLowerCase().split('\\');
+	const filename = paths.pop().replace(/_/g, '-') + '.php';
+	const folder = path.join(getProjectRoot(), getSetting('paths.php'), paths.join('/'));
+
+	writeFromTemplate(
+		folder,
+		`class-${filename}`,
+		`files/${template}.php`,
+		{
+			...getSetting(),
+			heading: header || filenameToHeading(filename) + ' template file',
+			description: description || 'Brief description of the styles in this file',
+			className: namespace.pop(),
+			namespace: namespace.length > 0 ? '\\' + namespace.join('\\') : '',
+		},
+		options.force
+	)
 }

package/src/commands/css.js

@@ -1,4 +1,3 @@
-
 /**
  * Node Dependencies
  */
@@ -7,7 +6,7 @@ import path from 'path';
 /**
  * Internal Dependencies
  */
-import { getSetting, writeFile, heading, filenameToHeading, getProjectRoot, msgErrorTitle, getTemplateFile, compileTemplate } from "../utilities/index.js";
+import { getSetting, heading, filenameToHeading, getProjectRoot, writeFromTemplate } from "../utilities/index.js";
 
 export default (name, options) => {
 	heading('Creating CSS file...')
@@ -16,21 +15,16 @@ export default (name, options) => {
 		d: description = null,
 	} = options
 
-	try {
-		const paths = name.toLowerCase().split('\\')
-		const filename = paths.pop() + '.css';
-		const folder = path.join(getProjectRoot(), getSetting('paths.css'), paths.join('/'));
-
-		// Data
-		const data = getSetting();
-		data.heading = header || filenameToHeading(name) + ' styles.';
-		data.description = description || 'Brief description of the styles in this file';
-
-		const content = compileTemplate(getTemplateFile('files/stylesheet.css'), data);
-		writeFile(folder, filename, content);
-	}
-	catch (err) {
-		msgErrorTitle('We failed!!!');
-		throw err;
-	}
+	const dest = path.join( getProjectRoot(), getSetting( 'paths.css' ), name.toLowerCase().replaceAll( '\\', '/' ) ) + '.css'
+	writeFromTemplate(
+		path.dirname( dest ),
+		path.basename( dest ),
+		'files/stylesheet.css',
+		{
+			...getSetting(),
+			heading: header || filenameToHeading(name) + ' styles.',
+			description: description || 'Brief description of the styles in this file',
+		},
+		options.force
+	)
 }

package/src/commands/init.js

@@ -2,13 +2,16 @@
  * External dependencies
  */
 import inquirer from 'inquirer'
-import kebabCase from 'lodash/kebabCase.js'
 
 /**
  * Node dependencies
  */
 import { getSetting, heading, saveConfig, msgSuccessTitle } from '../utilities/index.js'
 
+function kebabCase( value ) {
+	return String( value ).toLowerCase().replace( /\s+/g, '-' )
+}
+
 export default async () => {
     const questions = [
         // Product

package/src/commands/javascript.js

@@ -1,4 +1,3 @@
-
 /**
  * Node Dependencies
  */
@@ -7,7 +6,7 @@ import path from 'path';
 /**
  * Internal Dependencies
  */
-import { getSetting, writeFile, heading, filenameToHeading, getProjectRoot, msgErrorTitle, getTemplateFile, compileTemplate } from "../utilities/index.js";
+import { getSetting, heading, filenameToHeading, getProjectRoot, writeFromTemplate } from "../utilities/index.js";
 
 export default (name, options) => {
 	heading('Creating javascript file...')
@@ -16,21 +15,16 @@ export default (name, options) => {
 		d: description = null,
 	} = options
 
-	try {
-		const paths = name.toLowerCase().split('\\')
-		const filename = paths.pop() + '.js';
-		const folder = path.join(getProjectRoot(), getSetting('paths.javascript'), paths.join('/'));
-
-		// Data
-		const data = getSetting();
-        data.heading = header || filenameToHeading(name) + ' module.';
-		data.description = description || 'Description of the module';
-
-		const content = compileTemplate(getTemplateFile('files/javascript.js'), data);
-		writeFile(folder, filename, content);
-	}
-	catch (err) {
-		msgErrorTitle('We failed!!!');
-		throw err;
-	}
+	const dest = path.join( getProjectRoot(), getSetting( 'paths.javascript' ), name.toLowerCase().replaceAll( '\\', '/' ) ) + '.js'
+	writeFromTemplate(
+		path.dirname( dest ),
+		path.basename( dest ),
+		'files/javascript.js',
+		{
+			...getSetting(),
+			heading: header || filenameToHeading(name) + ' module.',
+			description: description || 'Description of the module',
+		},
+		options.force
+	)
 }