@wpmoo/toolkit 0.9.21 → 0.9.23

package/README.md

@@ -190,6 +190,17 @@ The unscoped `wpmoo` short alias is optional. If npm returns `E404` or rejects
 that alias during the publish workflow, the workflow reports a non-blocking
 warning while keeping the scoped package release valid.
 
+### Release candidate rules
+
+- **Required release artifacts**: `@wpmoo/toolkit`, `@wpmoo/odoo`, and
+  `@wpmoo/odoo-dev` must be publishable at package version `$VERSION` for tag
+  `v$VERSION` to count as valid.
+- **Optional artifact**: `wpmoo` at package version `$VERSION` is best-effort only. If npm
+  rejects it, the release still succeeds as long as the three required scoped
+  packages are valid.
+- **Smoke expectation**: run `npm run smoke:published -- "$VERSION"` after the
+  release tag workflow completes.
+
 ## Documentation
 
 - [External Resources](docs/external-resources.md)

package/dist/cli.js

@@ -38,7 +38,7 @@ import { confirmPrompt, introPrompt, isPromptCancel, notePrompt, outroPrompt, se
 import { renderBanner } from './templates.js';
 import { checkForUpdate, isUpdateCheckSkipped, restartCli } from './update-check.js';
 import { packageName, packageVersion, renderVersion, renderVersionTag } from './version.js';
-import { environmentStatusJson, getEnvironmentStatus, renderEnvironmentStatusForTarget, renderEnvironmentStatusSummary, } from './status.js';
+import { environmentStatusJson, getEnvironmentStatus, environmentBannerSummaryLine, renderEnvironmentStatusForTarget, } from './status.js';
 import { getGitHubAccounts, getGitHubRepositoryStatus, githubRepositoryUrl, realGitHub, createGitHubRepository, } from './github.js';
 import { environmentGitHubOwner } from './environment-context.js';
 import { handlePromptCancel, handleUnavailableMenuChoice, installPromptCancelKeyTracker, isMenuBackSignal, MenuBackSignal, menuIntroTitle, menuPromptMessage, } from './menu-navigation.js';
@@ -184,27 +184,16 @@ function validateRepoName(value) {
 function startupVersionLine(latestVersion) {
     return `v${packageVersion()}${latestVersion ? ` -> v${latestVersion} available` : ''}`;
 }
-function pluralize(count, singular, plural) {
-    return `${count} ${count === 1 ? singular : plural}`;
-}
-function renderStartupEnvironmentLine(status) {
-    if (status.kind !== 'environment') {
-        return `Environment: ${renderEnvironmentStatusSummary(status)}`;
-    }
-    const issueCount = status.composeErrors.length + status.invalidSourceRepoPaths.length + status.missingCoreFiles.length;
-    const issueSuffix = issueCount > 0 ? ` · ${pluralize(issueCount, 'issue', 'issues')}` : '';
-    return [
-        `Environment: Odoo ${status.odooVersion}`,
-        pluralize(status.sourceRepoCount, 'repo', 'repos'),
-        pluralize(status.moduleCandidateCount, 'module', 'modules'),
-    ].join(' · ') + issueSuffix;
-}
 function renderStartupBanner(details, latestVersion) {
     const versionLine = startupVersionLine(latestVersion);
     return renderBanner(details?.(versionLine), details ? { version: versionLine } : undefined);
 }
 function renderCockpitStatusLines(status, serviceStatus, lastStatus) {
-    return [renderStartupEnvironmentLine(status), renderServiceRuntimeStatusLine(serviceStatus), lastStatus];
+    return [
+        environmentBannerSummaryLine(status),
+        renderServiceRuntimeStatusLine(serviceStatus),
+        lastStatus,
+    ];
 }
 function renderLastCommandStatus(command) {
     return `Last: ${command.label} ✓ completed`;

package/dist/cockpit/command-registry.js

@@ -29,41 +29,40 @@ function internalCommand(id, category, label, description, aliases = []) {
     };
 }
 export const cockpitCommands = [
-    dailyCommand('start', 'services', 'Start services', 'Start the Odoo development services.', ['up', 'compose up']),
-    dailyCommand('stop', 'services', 'Stop services', 'Stop the Odoo development services.', ['down', 'compose down']),
-    dailyCommand('restart', 'services', 'Restart services', 'Restart the Odoo development services.', ['reload']),
-    dailyCommand('logs', 'services', 'View logs', 'Stream logs for an Odoo environment service.', ['log', 'tail']),
-    dailyCommand('shell', 'services', 'Open shell', 'Open a shell inside the Odoo service container.', ['bash', 'terminal']),
+    dailyCommand('start', 'services', 'Start services', 'Start Odoo services.', ['up', 'compose up']),
+    dailyCommand('stop', 'services', 'Stop services', 'Stop Odoo services.', ['down', 'compose down']),
+    dailyCommand('restart', 'services', 'Restart services', 'Restart Odoo services.', ['reload']),
+    dailyCommand('logs', 'services', 'View logs', 'Tail service logs.', ['log', 'tail']),
+    dailyCommand('shell', 'services', 'Open shell', 'Open a service shell.', ['bash', 'terminal']),
     internalCommand('list-modules', 'modules', 'List modules', 'Browse detected Odoo modules by source category.', [
         'modules list',
         'browse modules',
+        '/module',
+        'module',
     ]),
-    dailyCommand('install', 'modules', 'Install module', 'Install one or more Odoo modules into a database.', ['install module']),
-    dailyCommand('update', 'modules', 'Update module', 'Update one or more Odoo modules in a database.', ['upgrade']),
-    dailyCommand('test', 'modules', 'Run tests', 'Run Odoo tests for one or more modules.', ['tests', 'pytest']),
-    dailyCommand('lint', 'modules', 'Run environment lint', 'Run the configured environment lint checks.', ['check', 'quality']),
-    dailyCommand('pot', 'modules', 'Generate POT', 'Generate translation template files for a module.', ['translation', 'i18n']),
-    dailyCommand('psql', 'database', 'Open psql', 'Open a PostgreSQL prompt for an environment database.', ['postgres', 'sql']),
-    dailyCommand('snapshot', 'database', 'Create snapshot', 'Create a database snapshot.', ['backup', 'dump']),
-    dailyCommand('restore-snapshot', 'database', 'Restore snapshot', 'Restore a database from a named snapshot.', ['restore', 'snapshot restore']),
-    dailyCommand('resetdb', 'database', 'Reset database', 'Reset an environment database.', ['reset db', 'database reset']),
+    dailyCommand('install', 'modules', 'Install module', 'Install modules in the database.', ['install module', 'module']),
+    dailyCommand('update', 'modules', 'Update module', 'Update modules in the database.', ['upgrade', 'module']),
+    dailyCommand('test', 'modules', 'Run tests', 'Run tests for selected modules.', ['tests', 'pytest', 'module']),
+    dailyCommand('lint', 'modules', 'Run environment lint', 'Run environment lint checks.', ['check', 'quality']),
+    dailyCommand('pot', 'modules', 'Generate POT', 'Generate module translation templates.', ['translation', 'i18n']),
+    dailyCommand('psql', 'database', 'Open psql', 'Open PostgreSQL prompt.', ['postgres', 'sql', '/db']),
+    dailyCommand('snapshot', 'database', 'Create snapshot', 'Create a database snapshot.', ['backup', 'dump', '/snapshot']),
+    dailyCommand('restore-snapshot', 'database', 'Restore snapshot', 'Restore a named snapshot.', ['restore', 'snapshot restore']),
+    dailyCommand('resetdb', 'database', 'Reset database', 'Reset the environment database.', ['reset db', 'database reset']),
     internalCommand('status', 'diagnostics', 'Environment status', 'Show a summary of the current environment state.', [
         'state',
         'summary',
     ]),
-    internalCommand('doctor', 'diagnostics', 'Run doctor', 'Run environment diagnostics and report actionable issues.', [
-        'diagnose',
-        'health',
-    ]),
-    internalCommand('add-repo', 'repositories', 'Add source repo', 'Add a source repository as an environment submodule.', [
+    internalCommand('doctor', 'diagnostics', 'Run doctor', 'Run environment diagnostics.', ['diagnose', 'health']),
+    internalCommand('add-repo', 'repositories', 'Add source repo', 'Add a source repository.', [
         'repository add',
         'source add',
     ]),
-    internalCommand('remove-repo', 'repositories', 'Remove source repo', 'Remove a source repository from the environment.', ['repository remove', 'source remove']),
-    internalCommand('add-module', 'modules', 'Add module', 'Add a module folder to a source repository.', ['module add']),
-    internalCommand('remove-module', 'modules', 'Remove module', 'Remove a module folder from a source repository.', ['module remove']),
-    internalCommand('safe-reset', 'maintenance', 'Safe reset environment', 'Refresh generated environment files while preserving source repositories.', ['reset', 'refresh']),
-    internalCommand('exit', 'maintenance', 'Exit', 'Leave the command palette.', ['quit', 'back']),
+    internalCommand('remove-repo', 'repositories', 'Remove source repo', 'Remove a source repository.', ['repository remove', 'source remove']),
+    internalCommand('add-module', 'modules', 'Add module', 'Add a module to a source repository.', ['module add']),
+    internalCommand('remove-module', 'modules', 'Remove module', 'Remove a module from a source repository.', ['module remove']),
+    internalCommand('safe-reset', 'maintenance', 'Safe reset environment', 'Refresh generated files only.', ['reset', 'refresh', '/safe']),
+    internalCommand('exit', 'maintenance', 'Exit', 'Close the command palette.', ['quit', 'back']),
 ];
 const defaultCommandIds = new Set(['start', 'logs', 'test', 'status', 'doctor', 'exit']);
 export function normalizeCockpitSearchTerm(term) {

package/dist/cockpit/menu.js

@@ -33,6 +33,21 @@ function categoryHeading(category) {
 function commandName(command) {
     return `${rgb(226, 184, 96, ` ${command.label.padEnd(topLevelCommandLabelWidth)}`)}${dim(`  ${command.description}`)}`;
 }
+const disabledReasonNextStep = {
+    'No modules found.': 'Next: choose "Add module" first.',
+    'Services stopped.': 'Next: choose "Start services" first.',
+    'Already running.': 'Next: choose "Stop services" or "Restart services".',
+    'Docker not running.': 'Next: start Docker, then choose "Start services".',
+    'No source repos found.': 'Next: choose "Add source repo" first.',
+};
+function disabledError(reason) {
+    const base = 'This option is disabled and cannot be selected.';
+    if (!reason) {
+        return base;
+    }
+    const nextStep = disabledReasonNextStep[reason];
+    return nextStep ? `${base}\nReason: ${reason}\n${nextStep}` : `${base}\nReason: ${reason}`;
+}
 function serviceDisabledReason(command, serviceStatus) {
     if (command.category !== 'services' || !serviceStatus)
         return undefined;
@@ -56,9 +71,6 @@ function disabledReason(command, serviceStatus, moduleCount, sourceRepoCount) {
         moduleDisabledReason(command, moduleCount) ??
         sourceRepoDisabledReason(command, sourceRepoCount));
 }
-function disabledError() {
-    return 'This option is disabled and cannot be selected.';
-}
 function commandDisabledValue(reason) {
     if (!reason) {
         return undefined;
@@ -131,7 +143,7 @@ export async function selectCockpitTopLevelMenu(options = {}) {
         pageSize: topLevelPageSize(choices.length),
         loop: false,
         hideMessage: true,
-        disabledError: disabledError(),
+        disabledError,
         navigationWarning: options.navigationWarning,
         escapeBehavior: 'ignore',
     });

package/dist/prompts/index.js

@@ -171,10 +171,13 @@ function hiddenSelectTheme(disabledError, navigationHelp = 'exit', navigationWar
     };
 }
 function disabledErrorI18n(disabledError, activeReason) {
-    const i18n = { disabledError };
+    const i18n = typeof disabledError === 'string' ? { disabledError } : { disabledError: disabledError(undefined) };
     Object.defineProperty(i18n, 'disabledError', {
         get: () => {
             const reason = activeReason();
+            if (typeof disabledError === 'function') {
+                return disabledError(reason);
+            }
             return reason ? `${disabledError}\nReason: ${reason}` : disabledError;
         },
     });

package/dist/status.js

@@ -5,6 +5,10 @@ import { defaultOdooVersion, markerPath } from './environment.js';
 import { emptyModuleQualitySummary, mergeModuleQualitySummaries, scanModuleQuality, } from './module-quality.js';
 import { isValidPathSegment, validateRepoPath } from './path-validation.js';
 const validSourceTypes = ['private', 'oca', 'external'];
+const summarySeparator = ' \u00B7 ';
+function pluralize(count, singular, plural) {
+    return `${count} ${count === 1 ? singular : plural}`;
+}
 function normalizeSourceType(sourceType) {
     if (typeof sourceType === 'string' && validSourceTypes.includes(sourceType)) {
         return sourceType;
@@ -85,6 +89,18 @@ async function missingCoreFiles(target, odooVersion) {
     missing.push(...composeLayout.missingFiles);
     return { missing, composeFiles: composeLayout.files, composeErrors: composeLayout.errors };
 }
+export function environmentBannerSummaryLine(status) {
+    if (status.kind !== 'environment') {
+        return `Environment: ${summaryText(status)}`;
+    }
+    const issueCount = status.composeErrors.length + status.invalidSourceRepoPaths.length + status.missingCoreFiles.length;
+    const issueSuffix = issueCount > 0 ? `${summarySeparator}${pluralize(issueCount, 'issue', 'issues')}` : '';
+    return [
+        `Environment: Odoo ${status.odooVersion}`,
+        pluralize(status.sourceRepoCount, 'repo', 'repos'),
+        pluralize(status.moduleCandidateCount, 'module', 'modules'),
+    ].join(summarySeparator) + issueSuffix;
+}
 function summaryText(status) {
     if (status.kind === 'no_environment')
         return 'No WPMoo environment detected.';

package/docs/generated-environment-verification.md

@@ -160,3 +160,29 @@ npm test
 npm run test:coverage
 npm run build
 ```
+
+## Coverage watchlist (risk monitoring)
+
+The following list is a risk watchlist for Train 2 verification, not a hard gate.
+It uses the full `npm run test:coverage` suite to highlight where changes in
+high-impact runtime files should be reviewed with extra care:
+
+- `src/cli.ts`: **watch**: 83.74% line coverage (1458/1741), function coverage
+  92.47% (86/93), branch coverage 80.61% (420/521). This file remains the
+  highest-risk surface because it owns direct commands, cockpit dispatch, JSON
+  routes, and release-facing error behavior.
+- `src/doctor.ts`: **observe**: 94.48% line coverage (702/743), function
+  coverage 95.56% (43/45), branch coverage 86.42% (229/265).
+- `src/module-actions.ts`: **observe**: 96.83% line coverage (519/536),
+  function coverage 97.22% (35/36), branch coverage 88.97% (129/145).
+- `src/templates.ts`: **observe**: 99.24% line coverage (262/264), function
+  coverage 100.00% (38/38), branch coverage 90.08% (109/121).
+- `src/prompts/index.ts`: **observe**: 95.15% line coverage (294/309),
+  function coverage 100.00% (36/36), branch coverage 92.31% (96/104).
+
+Train 2 full-suite coverage baseline:
+
+- Statements: 92.65% (7304/7883)
+- Branches: 88.24% (2432/2756)
+- Functions: 96.27% (595/618)
+- Lines: 92.65% (7304/7883)

package/docs/handoff.md

@@ -42,7 +42,23 @@ npm view "@wpmoo/odoo-dev@$VERSION" version
 ```
 
 `npm view "wpmoo@$VERSION" version` is optional and may report that the short
-alias is absent.
+alias is absent. A release is valid when all required scoped packages verify:
+
+- `npm view "@wpmoo/toolkit@$VERSION" version`
+- `npm view "@wpmoo/odoo@$VERSION" version`
+- `npm view "@wpmoo/odoo-dev@$VERSION" version`
+
+Optional short alias rule:
+
+- `wpmoo` may be reported as missing or fail publish without invalidating the
+  release candidate. Scoped packages are the supported release artifacts and
+  are sufficient to mark the release valid.
+
+Suggested smoke check:
+
+```bash
+npm run smoke:published -- "$VERSION"
+```
 
 Current command standard:
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wpmoo/toolkit",
-  "version": "0.9.21",
+  "version": "0.9.23",
   "description": "WPMoo Toolkit for development, staging, and production lifecycle workflows.",
   "type": "module",
   "repository": {