@aifabrix/builder 2.52.1 → 2.53.1

package/.cursor/plans/Archive/157.1-security-quality-violations-by-type.csv

@@ -1,374 +0,0 @@
-rule_title,location,symbol_or_detail,message,done
-Comparison between inconvertible types,lib/app/certification-verify-rows.js:19,id,"Variable 'id' is of type date, object or regular expression, but it is compared to an expression of type null.",true
-Comparison between inconvertible types,lib/certification/merge-certification-from-artifact.js:16,id,"Variable 'id' is of type date, object or regular expression, but it is compared to an expression of type null.",true
-Comparison between inconvertible types,tests/local/lib/app-run-branch-coverage.test.js:232,error,"Variable 'error' cannot be of type null, but it is compared to an expression of type null.",true
-Identical operands,lib/app/show.js:480,,Operands configD ... Sources and configD ... Sources are identical.,true
-Identical operands,tests/lib/app/app-deploy.test.js:1159,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator-validation.test.js:232,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator-validation.test.js:285,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1372,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1376,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1416,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1420,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:145,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1450,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1462,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1492,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1525,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:155,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1561,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1596,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1626,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1656,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1686,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1716,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1750,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1756,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1792,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1798,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1835,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1841,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1870,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1882,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1909,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1935,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1963,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:1991,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:2019,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:2046,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:2080,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:2109,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:2124,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:2158,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:2192,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:2272,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:2288,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:2326,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:252,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:301,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:328,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:332,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:365,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:381,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/generator/generator.test.js:387,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/validation/validator.test.js:1008,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/validation/validator.test.js:1014,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/validation/validator.test.js:963,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Identical operands,tests/lib/validation/validator.test.js:969,,Operands filePat ... .yaml') and filePat ... .yaml') are identical.,true
-Missing await,lib/validation/validator.js:227,validateRoles,,true
-Unneeded defensive code,lib/infrastructure/helpers.js:184,,This guard always evaluates to false.,true
-Unneeded defensive code,lib/infrastructure/helpers.js:184,,This guard always evaluates to false.,true
-Unneeded defensive code,lib/utils/dev-config.js:48,,This guard always evaluates to false.,true
-Unneeded defensive code,lib/utils/dev-config.js:48,,This guard always evaluates to false.,true
-Unneeded defensive code,lib/utils/error-formatters/error-parser.js:28,,This guard always evaluates to false.,true
-Unneeded defensive code,lib/utils/error-formatters/error-parser.js:28,,This guard always evaluates to false.,true
-Unneeded defensive code,tests/lib/utils/token-manager.test.js:1072,,This guard always evaluates to false.,true
-Unneeded defensive code,tests/lib/utils/token-manager.test.js:1075,,This guard always evaluates to false.,true
-"Unused variable, import, function or class",tests/bin/aifabrix.test.js:117,,Unused variable mockSetupCommands.,true
-"Unused variable, import, function or class",tests/bin/aifabrix.test.js:118,,Unused variable mockSetupAppCommands.,true
-"Unused variable, import, function or class",tests/helpers/mock-controller.js:12,,Unused variable axios.,true
-"Unused variable, import, function or class",tests/integration/hubspot/hubspot-integration.test.js:19,,Unused variable validateAgainstSchema.,true
-"Unused variable, import, function or class",tests/integration/hubspot/hubspot-integration.test.js:25,,Unused variable generator.,true
-"Unused variable, import, function or class",tests/integration/steps/step-01-create.test.js:11,,Unused variable fs.,true
-"Unused variable, import, function or class",tests/integration/steps/step-02-dockerfile.test.js:24,,Unused variable port.,true
-"Unused variable, import, function or class",tests/integration/steps/step-03-resolve.test.js:23,,Unused variable port.,true
-"Unused variable, import, function or class",tests/integration/steps/step-04-json.test.js:23,,Unused variable port.,true
-"Unused variable, import, function or class",tests/integration/steps/step-06-build.test.js:21,,Unused variable port.,true
-"Unused variable, import, function or class",tests/integration/steps/step-09-validate-health.test.js:14,,Unused variable execCommand.,true
-"Unused variable, import, function or class",tests/integration/workflow.test.js:29,,Unused variable port.,true
-"Unused variable, import, function or class",tests/lib/api.test.js:33,,Unused variable getOrRefreshDeviceToken.,true
-"Unused variable, import, function or class",tests/lib/app/app-deploy.test.js:14,,Unused variable app.,true
-"Unused variable, import, function or class",tests/lib/app/app-helpers.test.js:383,,Unused variable variablesPath.,true
-"Unused variable, import, function or class",tests/lib/app/app-list.test.js:41,,Unused variable formatAuthenticationError.,true
-"Unused variable, import, function or class",tests/lib/app/app-run-helpers.test.js:105,,Unused variable readFile.,true
-"Unused variable, import, function or class",tests/lib/app/app-run-uncovered.test.js:179,,Unused variable config.,true
-"Unused variable, import, function or class",tests/lib/app/app-run-uncovered.test.js:209,,Unused variable execAsync.,true
-"Unused variable, import, function or class",tests/lib/app/app-run-uncovered.test.js:34,,Unused variable fs.,true
-"Unused variable, import, function or class",tests/lib/app/app.test.js:17,,Unused variable templates.,true
-"Unused variable, import, function or class",tests/lib/app/app.test.js:19,,Unused variable githubGenerator.,true
-"Unused variable, import, function or class",tests/lib/app/app.test.js:20,,Unused variable validator.,true
-"Unused variable, import, function or class",tests/lib/app/app.test.js:21,,Unused variable secrets.,true
-"Unused variable, import, function or class",tests/lib/app/app.test.js:485,,Unused variable execAsync.,true
-"Unused variable, import, function or class",tests/lib/app/show.test.js:62,,Unused variable fs.,true
-"Unused variable, import, function or class",tests/lib/app/show.test.js:75,,Unused variable loadVariablesFromPath.,true
-"Unused variable, import, function or class",tests/lib/cli-additional-branches.test.js:19,,Unused variable app.,true
-"Unused variable, import, function or class",tests/lib/cli-additional-branches.test.js:20,,Unused variable secrets.,true
-"Unused variable, import, function or class",tests/lib/cli-additional-branches.test.js:23,,Unused variable keyGenerator.,true
-"Unused variable, import, function or class",tests/lib/cli-additional-branches.test.js:24,,Unused variable handleCommandError.,true
-"Unused variable, import, function or class",tests/lib/cli-additional-branches.test.js:24,,Unused variable setupCommands.,true
-"Unused variable, import, function or class",tests/lib/cli-additional.test.js:20,,Unused variable secrets.,true
-"Unused variable, import, function or class",tests/lib/cli-additional.test.js:21,,Unused variable generator.,true
-"Unused variable, import, function or class",tests/lib/cli-additional.test.js:22,,Unused variable validator.,true
-"Unused variable, import, function or class",tests/lib/cli-additional.test.js:23,,Unused variable keyGenerator.,true
-"Unused variable, import, function or class",tests/lib/cli-additional.test.js:24,,Unused variable makeApiCall.,true
-"Unused variable, import, function or class",tests/lib/cli-branch-coverage.test.js:20,,Unused variable infra.,true
-"Unused variable, import, function or class",tests/lib/cli-branch-coverage.test.js:21,,Unused variable app.,true
-"Unused variable, import, function or class",tests/lib/cli-branch-coverage.test.js:22,,Unused variable secrets.,true
-"Unused variable, import, function or class",tests/lib/cli-branch-coverage.test.js:23,,Unused variable generator.,true
-"Unused variable, import, function or class",tests/lib/cli-branch-coverage.test.js:24,,Unused variable validator.,true
-"Unused variable, import, function or class",tests/lib/cli-branch-coverage.test.js:25,,Unused variable keyGenerator.,true
-"Unused variable, import, function or class",tests/lib/cli-branch-coverage.test.js:26,,Unused variable saveConfig.,true
-"Unused variable, import, function or class",tests/lib/cli-branch-coverage.test.js:27,,Unused variable makeApiCall.,true
-"Unused variable, import, function or class",tests/lib/cli-branch-coverage.test.js:28,,Unused variable inquirer.,true
-"Unused variable, import, function or class",tests/lib/cli-branch-coverage.test.js:29,,Unused variable exec.,true
-"Unused variable, import, function or class",tests/lib/cli-command-actions.test.js:24,,Unused variable validator.,true
-"Unused variable, import, function or class",tests/lib/cli-command-actions.test.js:25,,Unused variable keyGenerator.,true
-"Unused variable, import, function or class",tests/lib/cli-command-actions.test.js:36,,Unused variable exec.,true
-"Unused variable, import, function or class",tests/lib/cli-command-actions.test.js:684,,Unused variable options.,true
-"Unused variable, import, function or class",tests/lib/cli-command-actions.test.js:697,,Unused variable envPrompt.,true
-"Unused variable, import, function or class",tests/lib/cli-command-actions.test.js:72,,Unused variable method.,true
-"Unused variable, import, function or class",tests/lib/cli-commands.test.js:9,,Unused variable cli.,true
-"Unused variable, import, function or class",tests/lib/cli-comprehensive.test.js:106,,Unused variable keyGenerator.,true
-"Unused variable, import, function or class",tests/lib/cli-comprehensive.test.js:111,,Unused variable exec.,true
-"Unused variable, import, function or class",tests/lib/cli-coverage.test.js:15,,Unused variable keyGenerator.,true
-"Unused variable, import, function or class",tests/lib/cli-error-paths.test.js:34,,Unused variable validator.,true
-"Unused variable, import, function or class",tests/lib/cli-error-paths.test.js:36,,Unused variable logger.,true
-"Unused variable, import, function or class",tests/lib/cli-error-paths.test.js:40,,Unused variable chalk.,true
-"Unused variable, import, function or class",tests/lib/cli-setup.test.js:57,,Unused variable mockProgram.,true
-"Unused variable, import, function or class",tests/lib/cli-uncovered-commands.test.js:487,,Unused variable appPath.,true
-"Unused variable, import, function or class",tests/lib/cli.test.js:12,,Unused variable exec.,true
-"Unused variable, import, function or class",tests/lib/cli.test.js:13,,Unused variable net.,true
-"Unused variable, import, function or class",tests/lib/cli.test.js:143,,Unused variable mockSecretsPath.,true
-"Unused variable, import, function or class",tests/lib/cli.test.js:9,,Unused variable fs.,true
-"Unused variable, import, function or class",tests/lib/cli/setup-utility.test.js:99,,Unused variable schemaPath.,true
-"Unused variable, import, function or class",tests/lib/commands-app-branch-coverage.test.js:10,,Unused variable fsSync.,true
-"Unused variable, import, function or class",tests/lib/commands-app-direct.test.js:10,,Unused variable fsSync.,true
-"Unused variable, import, function or class",tests/lib/commands-app-direct.test.js:36,,Unused variable app.,true
-"Unused variable, import, function or class",tests/lib/commands-login.test.js:14,,Unused variable makeApiCall.,true
-"Unused variable, import, function or class",tests/lib/commands-login.test.js:882,,Unused variable validationResult.,true
-"Unused variable, import, function or class",tests/lib/commands/app-shell.test.js:47,,Unused variable p.,true
-"Unused variable, import, function or class",tests/lib/commands/app.test.js:12,,Unused variable yaml.,true
-"Unused variable, import, function or class",tests/lib/commands/convert.test.js:38,,Unused variable schemaBasePath.,true
-"Unused variable, import, function or class",tests/lib/commands/credential-env.test.js:146,,Unused variable result.,true
-"Unused variable, import, function or class",tests/lib/commands/credential-push.test.js:37,,Unused variable requireBearerForDataplanePipeline.,true
-"Unused variable, import, function or class",tests/lib/commands/dev-down.test.js:7,,Unused variable promisify.,true
-"Unused variable, import, function or class",tests/lib/commands/login-credentials.test.js:47,,Unused variable formatApiError.,true
-"Unused variable, import, function or class",tests/lib/commands/login-device.test.js:420,,Unused variable originalGetEnvironmentKey.,true
-"Unused variable, import, function or class",tests/lib/commands/login-device.test.js:60,,Unused variable ora.,true
-"Unused variable, import, function or class",tests/lib/commands/login-error-paths.test.js:24,,Unused variable logger.,true
-"Unused variable, import, function or class",tests/lib/commands/login-error-paths.test.js:25,,Unused variable tokenManager.,true
-"Unused variable, import, function or class",tests/lib/commands/logout.test.js:56,,Unused variable mockConfigPath.,true
-"Unused variable, import, function or class",tests/lib/commands/npm-config.test.js:32,,Unused variable path.,true
-"Unused variable, import, function or class",tests/lib/commands/repair-datasource.test.js:22,,Unused variable MINIMAL_METADATA_SCHEMA.,true
-"Unused variable, import, function or class",tests/lib/commands/repair.test.js:1874,,Unused variable result.,true
-"Unused variable, import, function or class",tests/lib/commands/repair.test.js:1886,,Unused variable dsPath.,true
-"Unused variable, import, function or class",tests/lib/commands/repair.test.js:265,,Unused variable result.,true
-"Unused variable, import, function or class",tests/lib/commands/secrets-list.test.js:6,,Unused variable path.,true
-"Unused variable, import, function or class",tests/lib/commands/secrets-remove.test.js:6,,Unused variable path.,true
-"Unused variable, import, function or class",tests/lib/commands/up-common.test.js:71,,Unused variable controllerUrlMod.,true
-"Unused variable, import, function or class",tests/lib/commands/upload.test.js:444,,Unused variable displayValidationResults.,true
-"Unused variable, import, function or class",tests/lib/commands/upload.test.js:9,,Unused variable chalk.,true
-"Unused variable, import, function or class",tests/lib/commands/wizard-core.test.js:64,,Unused variable datasourceDeploy.,true
-"Unused variable, import, function or class",tests/lib/commands/wizard-core.test.js:65,,Unused variable wizardDataplane.,true
-"Unused variable, import, function or class",tests/lib/commands/wizard.test.js:55,,Unused variable datasourceDeploy.,true
-"Unused variable, import, function or class",tests/lib/commands/wizard.test.js:56,,Unused variable wizardDataplane.,true
-"Unused variable, import, function or class",tests/lib/compose-generator.test.js:13,,Unused variable fs.,true
-"Unused variable, import, function or class",tests/lib/core/config.test.js:11,,Unused variable fs.,true
-"Unused variable, import, function or class",tests/lib/core/config.test.js:27,,Unused variable loadDeveloperId.,true
-"Unused variable, import, function or class",tests/lib/core/diff.test.js:10,,Unused variable path.,true
-"Unused variable, import, function or class",tests/lib/core/diff.test.js:11,,Unused variable chalk.,true
-"Unused variable, import, function or class",tests/lib/core/diff.test.js:9,,Unused variable fs.,true
-"Unused variable, import, function or class",tests/lib/core/key-generator.test.js:278,,Unused variable variablesPath.,true
-"Unused variable, import, function or class",tests/lib/core/key-generator.test.js:47,,Unused variable variablesPath.,true
-"Unused variable, import, function or class",tests/lib/core/secrets-utils.test.js:376,,Unused variable buildSecrets.,true
-"Unused variable, import, function or class",tests/lib/core/secrets-utils.test.js:682,,Unused variable serviceVariablesPath.,true
-"Unused variable, import, function or class",tests/lib/core/secrets-utils.test.js:693,,Unused variable serviceVariablesPath.,true
-"Unused variable, import, function or class",tests/lib/core/secrets-utils.test.js:703,,Unused variable serviceVariablesPath.,true
-"Unused variable, import, function or class",tests/lib/core/secrets-utils.test.js:722,,Unused variable serviceVariablesPath.,true
-"Unused variable, import, function or class",tests/lib/core/secrets-utils.test.js:738,,Unused variable serviceVariablesPath.,true
-"Unused variable, import, function or class",tests/lib/core/secrets-utils.test.js:754,,Unused variable serviceVariablesPath.,true
-"Unused variable, import, function or class",tests/lib/core/secrets-utils.test.js:772,,Unused variable serviceVariablesPath.,true
-"Unused variable, import, function or class",tests/lib/core/secrets-utils.test.js:792,,Unused variable serviceVariablesPath.,true
-"Unused variable, import, function or class",tests/lib/core/secrets.test.js:1022,,Unused variable mockSecrets.,true
-"Unused variable, import, function or class",tests/lib/core/secrets.test.js:114,,Unused variable mockEnvConfig.,true
-"Unused variable, import, function or class",tests/lib/core/secrets.test.js:2417,,Unused variable builderPath.,true
-"Unused variable, import, function or class",tests/lib/core/secrets.test.js:2761,,Unused variable result.,true
-"Unused variable, import, function or class",tests/lib/core/secrets.test.js:3247,,Unused variable secretsPath.,true
-"Unused variable, import, function or class",tests/lib/core/secrets.test.js:3274,,Unused variable secretsPath.,true
-"Unused variable, import, function or class",tests/lib/core/secrets.test.js:3779,,Unused variable outputPath.,true
-"Unused variable, import, function or class",tests/lib/core/secrets.test.js:3854,,Unused variable builderPath.,true
-"Unused variable, import, function or class",tests/lib/core/secrets.test.js:3982,,Unused variable envPath.,true
-"Unused variable, import, function or class",tests/lib/core/secrets.test.js:709,,Unused variable mockSecrets.,true
-"Unused variable, import, function or class",tests/lib/datasource/datasource-deploy.test.js:9,,Unused variable fs.,true
-"Unused variable, import, function or class",tests/lib/datasource/datasource-diff.test.js:9,,Unused variable chalk.,true
-"Unused variable, import, function or class",tests/lib/datasource/datasource-list.test.js:9,,Unused variable chalk.,true
-"Unused variable, import, function or class",tests/lib/datasource/datasource-validate.test.js:9,,Unused variable fs.,true
-"Unused variable, import, function or class",tests/lib/datasource/log-viewer.test.js:154,,Unused variable path.,true
-"Unused variable, import, function or class",tests/lib/deployment/deployer-error-paths.test.js:18,,Unused variable axios.,true
-"Unused variable, import, function or class",tests/lib/deployment/deployer-error-paths.test.js:20,,Unused variable logger.,true
-"Unused variable, import, function or class",tests/lib/deployment/deployer-error-paths.test.js:24,,Unused variable chalk.,true
-"Unused variable, import, function or class",tests/lib/deployment/environment-deploy.test.js:10,,Unused variable chalk.,true
-"Unused variable, import, function or class",tests/lib/deployment/environment-deploy.test.js:101,,Unused variable environmentsApi.,true
-"Unused variable, import, function or class",tests/lib/deployment/push-error-paths.test.js:103,,Unused variable errorOutput.,true
-"Unused variable, import, function or class",tests/lib/deployment/push-error-paths.test.js:11,,Unused variable chalk.,true
-"Unused variable, import, function or class",tests/lib/deployment/push-error-paths.test.js:12,,Unused variable logger.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-deploy-helpers.test.js:12,,Unused variable yaml.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-deploy-helpers.test.js:202,,Unused variable fallbackPath1.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-deploy-helpers.test.js:302,,Unused variable datasourcePath2.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-deploy.test.js:10,,Unused variable fsPromises.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-deploy.test.js:12,,Unused variable yaml.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-deploy.test.js:128,,Unused variable variablesPath.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-deploy.test.js:13,,Unused variable chalk.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-deploy.test.js:9,,Unused variable fs.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-download.test.js:110,,Unused variable tempDir.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-download.test.js:12,,Unused variable chalk.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-generator.test.js:13,,Unused variable chalk.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-generator.test.js:14,,Unused variable handlebars.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-generator.test.js:793,,Unused variable datasourcePaths.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-test-error-paths.test.js:51,,Unused variable yaml.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-test-helpers.test.js:10,,Unused variable logger.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-test-helpers.test.js:12,,Unused variable externalSystemSchema.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-test-helpers.test.js:243,,Unused variable datasourceSchema.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-test-helpers.test.js:9,,Unused variable chalk.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-test.test.js:10,,Unused variable yaml.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-test.test.js:100,,Unused variable validateAgainstSchema.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-test.test.js:103,,Unused variable displayTestResults.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-test.test.js:104,,Unused variable displayIntegrationTestResults.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-test.test.js:11,,Unused variable chalk.,true
-"Unused variable, import, function or class",tests/lib/external-system/external-system-test.test.js:110,,Unused variable systemKey.,true
-"Unused variable, import, function or class",tests/lib/generator/external-controller-manifest.test.js:11,,Unused variable yaml.,true
-"Unused variable, import, function or class",tests/lib/generator/external-controller-manifest.test.js:368,,Unused variable generateControllerManifest.,true
-"Unused variable, import, function or class",tests/lib/generator/external-controller-manifest.test.js:40,,Unused variable variablesPath.,true
-"Unused variable, import, function or class",tests/lib/generator/generator-error-paths.test.js:189,,Unused variable variablesPath.,true
-"Unused variable, import, function or class",tests/lib/generator/generator-error-paths.test.js:284,,Unused variable variablesPath.,true
-"Unused variable, import, function or class",tests/lib/generator/generator-error-paths.test.js:339,,Unused variable systemFilePath.,true
-"Unused variable, import, function or class",tests/lib/generator/generator-error-paths.test.js:358,,Unused variable systemFilePath.,true
-"Unused variable, import, function or class",tests/lib/generator/generator-helpers.test.js:10,,Unused variable path.,true
-"Unused variable, import, function or class",tests/lib/generator/generator-helpers.test.js:11,,Unused variable yaml.,true
-"Unused variable, import, function or class",tests/lib/generator/generator-validation.test.js:16,,Unused variable keyGenerator.,true
-"Unused variable, import, function or class",tests/lib/generator/generator.test.js:13,,Unused variable keyGenerator.,true
-"Unused variable, import, function or class",tests/lib/generator/generator.test.js:2547,,Unused variable missingPath.,true
-"Unused variable, import, function or class",tests/lib/generator/generator.test.js:264,,Unused variable result.,true
-"Unused variable, import, function or class",tests/lib/generator/generator.test.js:306,,Unused variable result.,true
-"Unused variable, import, function or class",tests/lib/generator/generator.test.js:63,,Unused variable variablesPath.,true
-"Unused variable, import, function or class",tests/lib/generator/generator.test.js:65,,Unused variable rbacPath.,true
-"Unused variable, import, function or class",tests/lib/generator/wizard-generator.test.js:70,,Unused variable yaml.,true
-"Unused variable, import, function or class",tests/lib/health-check.test.js:10,,Unused variable exec.,true
-"Unused variable, import, function or class",tests/lib/infrastructure/infra.test.js:136,,Unused variable mockTempComposePath.,true
-"Unused variable, import, function or class",tests/lib/programmatic/openapi-description-enricher.test.js:19,,Unused variable name.,true
-"Unused variable, import, function or class",tests/lib/utils/api.test.js:36,,Unused variable getOrRefreshDeviceToken.,true
-"Unused variable, import, function or class",tests/lib/utils/app-register-config.test.js:41,,Unused variable yaml.,true
-"Unused variable, import, function or class",tests/lib/utils/app-register-validator.test.js:22,,Unused variable originalRelative.,true
-"Unused variable, import, function or class",tests/lib/utils/compose-generate-infra-pgpass.test.js:26,,Unused variable sys.,true
-"Unused variable, import, function or class",tests/lib/utils/configuration-env-resolver.test.js:279,,Unused variable envPath.,true
-"Unused variable, import, function or class",tests/lib/utils/controller-url.test.js:15,,Unused variable isControllerHealthReachable.,true
-"Unused variable, import, function or class",tests/lib/utils/credential-display.test.js:20,,Unused variable STATUS_LABELS.,true
-"Unused variable, import, function or class",tests/lib/utils/dev-cert-helper.test.js:8,,Unused variable os.,true
-"Unused variable, import, function or class",tests/lib/utils/docker.test.js:30,,Unused variable execAsync.,true
-"Unused variable, import, function or class",tests/lib/utils/env-generation.test.js:1839,,Unused variable outEnvPath.,true
-"Unused variable, import, function or class",tests/lib/utils/env-generation.test.js:1882,,Unused variable outEnvPath.,true
-"Unused variable, import, function or class",tests/lib/utils/env-ports.test.js:196,,Unused variable parsed.,true
-"Unused variable, import, function or class",tests/lib/utils/error-formatters/error-parser.test.js:91,,Unused variable errorData.,true
-"Unused variable, import, function or class",tests/lib/utils/external-env-template.test.js:6,,Unused variable path.,true
-"Unused variable, import, function or class",tests/lib/utils/external-env-template.test.js:7,,Unused variable fs.,true
-"Unused variable, import, function or class",tests/lib/utils/external-system-display.test.js:9,,Unused variable chalk.,true
-"Unused variable, import, function or class",tests/lib/utils/external-system-readiness-core.test.js:11,,Unused variable summarizeDatasourceTiersA.,true
-"Unused variable, import, function or class",tests/lib/utils/file-upload.test.js:10,,Unused variable path.,true
-"Unused variable, import, function or class",tests/lib/utils/local-secrets.test.js:168,,Unused variable secretsDir.,true
-"Unused variable, import, function or class",tests/lib/utils/local-secrets.test.js:29,,Unused variable yaml.,true
-"Unused variable, import, function or class",tests/lib/utils/local-secrets.test.js:30,,Unused variable logger.,true
-"Unused variable, import, function or class",tests/lib/utils/local-secrets.test.js:69,,Unused variable secretsPath.,true
-"Unused variable, import, function or class",tests/lib/utils/mutagen-install.test.js:13,,Unused variable fs.,true
-"Unused variable, import, function or class",tests/lib/utils/mutagen-install.test.js:15,,Unused variable exec.,true
-"Unused variable, import, function or class",tests/lib/utils/mutagen-install.test.js:6,,Unused variable path.,true
-"Unused variable, import, function or class",tests/lib/utils/paths-detect-app-type.test.js:19,,Unused variable fs.,true
-"Unused variable, import, function or class",tests/lib/utils/paths.test.js:66,,Unused variable canUnsetJestWorker.,true
-"Unused variable, import, function or class",tests/lib/utils/resolve-application-config-path.test.js:19,,Unused variable fs.,true
-"Unused variable, import, function or class",tests/lib/utils/run-detect-app-type.js:6,,Unused variable path.,true
-"Unused variable, import, function or class",tests/lib/utils/run-get-resolve-app-path.js:6,,Unused variable path.,true
-"Unused variable, import, function or class",tests/lib/utils/schema-resolver.test.js:276,,Unused variable schemaBasePath.,true
-"Unused variable, import, function or class",tests/lib/utils/schema-resolver.test.js:31,,Unused variable mockSchemaBasePath.,true
-"Unused variable, import, function or class",tests/lib/utils/schema-resolver.test.js:9,,Unused variable fs.,true
-"Unused variable, import, function or class",tests/lib/utils/secrets-generator.test.js:14,,Unused variable crypto.,true
-"Unused variable, import, function or class",tests/lib/utils/secrets-generator.test.js:836,,Unused variable resolvedPath.,true
-"Unused variable, import, function or class",tests/lib/utils/secrets-generator.test.js:881,,Unused variable dir.,true
-"Unused variable, import, function or class",tests/lib/utils/secrets-path.test.js:12,,Unused variable yaml.,true
-"Unused variable, import, function or class",tests/lib/utils/secrets-url.test.js:114,,Unused variable result.,true
-"Unused variable, import, function or class",tests/lib/utils/secrets-url.test.js:141,,Unused variable result.,true
-"Unused variable, import, function or class",tests/lib/utils/secrets-url.test.js:177,,Unused variable result.,true
-"Unused variable, import, function or class",tests/lib/utils/secrets-url.test.js:85,,Unused variable result.,true
-"Unused variable, import, function or class",tests/lib/utils/ssh-key-helper.test.js:82,,Unused variable ed25519Path.,true
-"Unused variable, import, function or class",tests/lib/utils/template-helpers.test.js:105,,Unused variable configPath.,true
-"Unused variable, import, function or class",tests/lib/utils/template-helpers.test.js:89,,Unused variable invalidContent.,true
-"Unused variable, import, function or class",tests/lib/utils/token-manager.test.js:590,,Unused variable afterCall.,true
-"Unused variable, import, function or class",tests/lib/utils/token-manager.test.js:622,,Unused variable afterCall.,true
-"Unused variable, import, function or class",tests/lib/utils/token-manager.test.js:66,,Unused variable mockSecretsPath.,true
-"Unused variable, import, function or class",tests/lib/utils/yaml-preserve.test.js:15,,Unused variable isEncrypted.,true
-"Unused variable, import, function or class",tests/lib/validation/openapi-contract-surface-validator.test.js:7,,Unused variable fs.,true
-"Unused variable, import, function or class",tests/lib/validation/template.test.js:103,,Unused variable templatePath.,true
-"Unused variable, import, function or class",tests/lib/validation/template.test.js:117,,Unused variable templatePath.,true
-"Unused variable, import, function or class",tests/lib/validation/template.test.js:229,,Unused variable subdirPath.,true
-"Unused variable, import, function or class",tests/lib/validation/template.test.js:393,,Unused variable languageTemplatePath.,true
-"Unused variable, import, function or class",tests/lib/validation/template.test.js:537,,Unused variable templatesDir.,true
-"Unused variable, import, function or class",tests/lib/validation/template.test.js:90,,Unused variable templatePath.,true
-"Unused variable, import, function or class",tests/lib/validation/validate.test.js:11,,Unused variable chalk.,true
-"Unused variable, import, function or class",tests/lib/validation/validate.test.js:9,,Unused variable fs.,true
-"Unused variable, import, function or class",tests/lib/validation/validator-external-rbac.test.js:203,,Unused variable result.,true
-"Unused variable, import, function or class",tests/lib/validation/validator.test.js:149,,Unused variable mockSecretsPath.,true
-"Unused variable, import, function or class",tests/lib/validation/validator.test.js:183,,Unused variable variablesPath.,true
-"Unused variable, import, function or class",tests/lib/validation/validator.test.js:186,,Unused variable validVariables.,true
-"Unused variable, import, function or class",tests/lib/validation/validator.test.js:216,,Unused variable invalidVariables.,true
-"Unused variable, import, function or class",tests/lib/validation/validator.test.js:313,,Unused variable rbacPath.,true
-"Unused variable, import, function or class",tests/lib/validation/validator.test.js:945,,Unused variable validVariables.,true
-"Unused variable, import, function or class",tests/lib/validation/validator.test.js:955,,Unused variable validRbac.,true
-"Unused variable, import, function or class",tests/local/lib/app-commands.test.js:102,,Unused variable options.,true
-"Unused variable, import, function or class",tests/local/lib/app-commands.test.js:125,,Unused variable response.,true
-"Unused variable, import, function or class",tests/local/lib/app-commands.test.js:134,,Unused variable appName.,true
-"Unused variable, import, function or class",tests/local/lib/app-commands.test.js:135,,Unused variable environment.,true
-"Unused variable, import, function or class",tests/local/lib/app-commands.test.js:174,,Unused variable environment.,true
-"Unused variable, import, function or class",tests/local/lib/app-commands.test.js:248,,Unused variable environment.,true
-"Unused variable, import, function or class",tests/local/lib/app-commands.test.js:257,,Unused variable variables.,true
-"Unused variable, import, function or class",tests/local/lib/app-commands.test.js:331,,Unused variable environment.,true
-"Unused variable, import, function or class",tests/local/lib/app-commands.test.js:99,,Unused variable appCommands.,true
-"Unused variable, import, function or class",tests/local/lib/app-run-advanced.test.js:108,,Unused variable exec.,true
-"Unused variable, import, function or class",tests/local/lib/app-run-advanced.test.js:110,,Unused variable net.,true
-"Unused variable, import, function or class",tests/local/lib/app-run-advanced.test.js:299,,Unused variable execAsync.,true
-"Unused variable, import, function or class",tests/local/lib/app-run-advanced.test.js:311,,Unused variable execAsync.,true
-"Unused variable, import, function or class",tests/local/lib/app-run-advanced.test.js:323,,Unused variable execAsync.,true
-"Unused variable, import, function or class",tests/local/lib/app-run-advanced.test.js:94,,Unused variable language.,true
-"Unused variable, import, function or class",tests/local/lib/app-run-debug.test.js:163,,Unused variable getProjectRoot.,true
-"Unused variable, import, function or class",tests/local/lib/app-uncovered-paths.test.js:139,,Unused variable config.,true
-"Unused variable, import, function or class",tests/local/lib/app-uncovered-paths.test.js:323,,Unused variable updatedVariables.,true
-"Unused variable, import, function or class",tests/local/lib/app-uncovered-paths.test.js:50,,Unused variable getProjectRoot.,true
-"Unused variable, import, function or class",tests/local/lib/app/app-additional-coverage.test.js:42,,Unused variable build.,true
-"Unused variable, import, function or class",tests/local/lib/app/app-additional-coverage.test.js:43,,Unused variable appRun.,true
-"Unused variable, import, function or class",tests/local/lib/app/app-coverage-uncovered.test.js:155,,Unused variable config.,true
-"Unused variable, import, function or class",tests/local/lib/app/app-coverage-uncovered.test.js:191,,Unused variable config.,true
-"Unused variable, import, function or class",tests/local/lib/app/app-uncovered-lines.test.js:16,,Unused variable yaml.,true
-"Unused variable, import, function or class",tests/local/lib/commands-app.test.js:146,,Unused variable variablesPath.,true
-"Unused variable, import, function or class",tests/local/lib/commands-app.test.js:183,,Unused variable environment.,true
-"Unused variable, import, function or class",tests/local/lib/commands-app.test.js:218,,Unused variable appKey.,true
-"Unused variable, import, function or class",tests/local/lib/commands-app.test.js:219,,Unused variable environment.,true
-"Unused variable, import, function or class",tests/local/lib/commands-app.test.js:83,,Unused variable result.,true
-"Unused variable, import, function or class",tests/local/lib/commands-app.test.js:93,,Unused variable environment.,true
-"Unused variable, import, function or class",tests/local/lib/resolvers/dimension-file.test.js:13,,Unused variable mkdirSync.,true
-"Unused variable, import, function or class",tests/local/lib/utils/external-readme.test.js:10,,Unused variable fs.,true
-"Unused variable, import, function or class",tests/local/lib/utils/external-readme.test.js:11,,Unused variable handlebars.,true
-"Unused variable, import, function or class",tests/local/lib/utils/external-readme.test.js:174,,Unused variable originalGenerate.,true
-"Unused variable, import, function or class",tests/local/lib/utils/external-readme.test.js:9,,Unused variable path.,true
-"Unused variable, import, function or class",tests/scripts/test-wrapper.js:51,,Unused function filterExitHandlerError.,true
-"Unused variable, import, function or class",tests/setup.js:164,,Unused variable originalMkdirSync.,true
-"Unused variable, import, function or class",tests/setup.js:165,,Unused variable originalExistsSync.,true
-Useless assignment to local variable,lib/commands/setup.js:294,,"The initial value of outcome is unused, since it is always overwritten.",true
-Useless assignment to local variable,lib/core/audit-logger.js:241,,"The initial value of controllerUrl is unused, since it is always overwritten.",true
-Useless assignment to local variable,lib/utils/external-env-template.js:145,,"The initial value of authMethod is unused, since it is always overwritten.",true
-Useless assignment to local variable,lib/utils/manifest-source-emit.js:77,,"The initial value of tier is unused, since it is always overwritten.",true
-Useless assignment to local variable,scripts/pnpm/lib/resolve-aifabrix-bin.mjs:95,,"The initial value of resolved is unused, since it is always overwritten.",true
-Useless assignment to local variable,tests/lib/app/app-run-helpers.test.js:89,,The value assigned to opts here is unused.,true
-Useless assignment to local variable,tests/lib/infrastructure/infra.test.js:162,,The value assigned to options here is unused.,true
-Useless assignment to local variable,tests/lib/infrastructure/infra.test.js:211,,The value assigned to options here is unused.,true
-Useless assignment to local variable,tests/lib/infrastructure/infra.test.js:300,,The value assigned to options here is unused.,true
-Useless assignment to local variable,tests/lib/infrastructure/infra.test.js:331,,The value assigned to options here is unused.,true
-Useless assignment to local variable,tests/lib/infrastructure/infra.test.js:368,,The value assigned to options here is unused.,true
-Useless assignment to local variable,tests/lib/infrastructure/infra.test.js:504,,The value assigned to options here is unused.,true
-Useless assignment to local variable,tests/lib/infrastructure/infra.test.js:548,,The value assigned to options here is unused.,true
-Useless assignment to local variable,tests/lib/infrastructure/services.test.js:105,,The value assigned to opts here is unused.,true
-Useless assignment to local variable,tests/lib/infrastructure/services.test.js:47,,The value assigned to opts here is unused.,true
-Useless assignment to property,tests/lib/infrastructure/infra.test.js:578,checkInfraHealth,,true
-Useless conditional,integration/hubspot-test/create-hubspot.js:339,filePatterns,This use of variable 'filePatterns' always evaluates to false.,true
-Useless conditional,lib/app/show.js:350,controllerUrl,This use of variable 'controllerUrl' always evaluates to false.,true
-Useless conditional,lib/commands/repair-openapi-sync.js:100,res,This use of variable 'res' always evaluates to true.,true
-Useless conditional,lib/core/secrets-load.js:136,keySources,This use of variable 'keySources' always evaluates to true.,true
-Useless conditional,lib/datasource/capability/relate-operations.js:26,list,This use of variable 'list' always evaluates to true.,true
-Useless conditional,lib/utils/env-copy.js:390,variablesPath,This use of variable 'variablesPath' always evaluates to true.,true
-Useless conditional,scripts/lib/plan-parse.js:114,,This call to parseSimpleYaml always evaluates to true.,true

package/.cursor/process/README.md

@@ -1,9 +0,0 @@
-# Process documentation
-
-Canonical process descriptions for aifabrix-builder automation and release workflows.
-
-| Process | Description |
-| ------- | ----------- |
-| [deployment.md](./deployment.md) | NPM release: preflight, release branch CI, PR to `main`, publish and GitHub release |
-
-Command specs (agent-executable) live under [`.cursor/commands/`](../commands/).

package/.cursor/rules/anchor-docs.mdc

@@ -1,15 +0,0 @@
----
-description: Mandatory structure for Builder CLI anchor-docs.
-globs: anchor-docs/**/*.md
-alwaysApply: false
----
-
-# anchor-docs (Builder)
-
-When editing `anchor-docs/**/*.md` (only `README.md` is exempt). Copy `anchor-docs/_TEMPLATE` to create a new anchor (no `.md` extension — not validated):
-
-- Include YAML frontmatter: `title`, `navigation_id`, `owner`, `last_verified_commit`.
-- `navigation_id` must match **aifabrix-docs** `docs/builder/navigation.yaml` entries (`builder/<url>`).
-- Sections required: `## Summary`, `## Details`, `## Examples`, `## Paths`.
-- Reference repo files with **`[path:aifabrix-builder/...]`**; avoid backtick-only paths for the same references.
-- Align with CLI user-doc rules: keep HTTP/API endpoint names out of `docs/commands/` user text; technical anchors may cite code paths.

package/.cursor/rules/assistant-driven-integration-build.mdc

@@ -1,40 +0,0 @@
----
-description: >-
-  Apply when Builder CLI, builder-api, or agents participate in assistant-driven
-  Connected System builds. Product-neutral; runtime-orchestration is the agent
-  Work MCP; no vendor special-cases in lib/.
-globs:
-  - lib/commands/**/*
-  - lib/cli/**/*
-  - packages/builder-api/**/*
-  - builder/builder-api/**/*
-  - docs/commands/**/*
-alwaysApply: false
----
-
-# Assistant-driven integration build (Builder)
-
-**Dataplane SSOT (full law):**  
-`../aifabrix-dataplane/.cursor/rules/dev-rules/assistant-driven-integration-build.md`  
-(in monorepo layouts use the dataplane sibling path; do not fork the law text here.)
-
-## Builder / CLI role
-
-| Mode | Allowed |
-| --- | --- |
-| **Prep** | `validate` / `upload` / `repair` for **platform** apps (builder-api), evidence import, diagnose |
-| **Proof** | Not primary — Role Assistant drives published builder-api via dataplane Runtime MCP |
-| **Improve product** | Generic builder-api / CLI gaps for **any** `systemKey` — never HubSpot/lab branches in `lib/` |
-
-## Non-negotiable
-
-1. Obey [cli-product-neutral.mdc](cli-product-neutral.mdc) — no lab keys or plan IDs in product paths.
-2. Agents proving vertical builds must orchestrate via dataplane **`runtime-orchestration`** MCP, not by inventing CLI greenfield as the proof.
-3. Fix Builder when assistant proof shows a **generic** Builder bug; do not paper over with dataplane vendor shortcuts.
-4. Dataplane capability execute binds **via COM only** — see dataplane `no-direct-communication-only-via-com.md` (no raw CIP digs in Runtime).
-
-## Related
-
-- Epic: `aifabrix-dataplane/.cursor/plans/2. worker/473.0-business-transformation-assistant.plan.md`
-- Dataplane hardcoding law: `aifabrix-dataplane/.cursor/rules/dev-rules/no-integration-hardcoding-in-app.md`
-- Dataplane COM binding law: `aifabrix-dataplane/.cursor/rules/dev-rules/no-direct-communication-only-via-com.md`

package/.cursor/rules/cli-layout.mdc

@@ -1,81 +0,0 @@
----
-description: Apply when adding or changing aifabrix CLI commands, subcommands, flags, help text, terminal output (chalk), TTY vs JSON/--json, or CLI tests. Ensures layout, glyphs, and output profiles match repo standards.
-alwaysApply: false
----
-
-# Builder CLI layout rules
-
-When you touch **CLI surface** (`bin/aifabrix.js`, `lib/cli.js`, `lib/cli/**`, `lib/commands/**`) or **shared CLI formatters**, you must keep behavior aligned with the canonical docs in this folder.
-
-**Product-neutral copy:** User-facing help, errors, and warnings must not reference internal plan IDs or hardcode lab-only integration names in generic code. See [cli-product-neutral.mdc](./cli-product-neutral.mdc).
-
-## Sources of truth (read before shipping)
-
-| Document | Role |
-| -------- | ---- |
-| [layout.md](./layout.md) | Visual and semantic spec: colors, sections, glyphs, blocking vs warning, non-TTY/CI, helper map to `cli-test-layout-chalk.js`. |
-| [cli-output-command-matrix.md](./cli-output-command-matrix.md) | **One row per leaf command**: expected **output profile** (layout-blocks, tty-summary, stream-logs, json-opt, stdout-only, delegate) and **Manifest roots (141)** (when to emit gray manifest path line — see plan 141). |
-
-If implementation and `layout.md` disagree, **fix implementation** or **update docs** in the same change so they stay one story.
-
-## Output profiles (from the matrix)
-
-Use the matrix row for your command to decide UX:
-
-- **layout-blocks** — Structured blocks (header, verdict, lists, failures). Prefer helpers from `lib/utils/cli-test-layout-chalk.js` (or `lib/utils/cli-layout-chalk.js` re-export). Respect **`--json`**: skip TTY-only layout when the command is classified **json-opt** or when JSON mode is documented for that flow.
-- **tty-summary** — Short chalk lines / tables; still use canonical glyphs and semantic colors.
-- **stream-logs** — Tail/stream style; do not force layout-blocks around raw log lines unless product asks for a wrapper.
-- **json-opt** — Human TTY layout when interactive; **machine-oriented** output when `--json` (no decorative layout).
-- **stdout-only** — Stable, script-friendly stdout; minimal chalk; often used for diffs or piping.
-- **delegate** — Output comes from delegated library (wizard, etc.); still avoid forbidden glyphs and spurious decoration in anything you add.
-
-**New leaf command:** add a row to [cli-output-command-matrix.md](./cli-output-command-matrix.md) with the correct profile combination and **Manifest roots (141)** code (`141`, `141+`, `int`, `cfg`, `—`).
-
-## Manifest path visibility (plan 141)
-
-When a command’s matrix cell is **141**, **141+**, or **int** and the implementation reads an `application.yaml` (or integration equivalent), print **one** gray metadata line for humans (e.g. `Manifest: cwd/builder — /abs/path/application.yaml`) using `metadata()` or a small helper from `cli-test-layout-chalk.js`. Do not spam per-substep. Follow [.cursor/plans/141-manifest-location.plan.md](../plans/141-manifest-location.plan.md).
-
-**`aifabrix setup`:** destructive / conflict prompts that list folders to be replaced must use **resolved absolute paths** for every directory (plan § Guided setup — not only `builder/<app>/` under an otherwise absolute builder root).
-
-From [layout.md](./layout.md) contributor appendix:
-
-| Role | Use | Do not use |
-| ---- | --- | ----------- |
-| Success | **✔** | ✓, heavy check, decorative check emoji |
-| Warning | **⚠** | ‼, random emoji |
-| Failure | **✖** | ✗, `x`, decorative X |
-| Skipped | **⏭** | — |
-| In progress | **⏳** | Hourglass as pure decoration |
-
-- **Red = blocking only** (hard failure / non-zero exit when that line is the verdict). **Yellow = non-blocking** warnings. Never use red for warnings.
-- **Non-TTY / CI** (`NO_COLOR`, chalk level 0): **same glyphs and line structure**; colors become no-ops. Do not invent a second plain-text vocabulary (e.g. `[OK]` instead of ✔) unless the command is explicitly **stdout-only machine** output.
-
-Markdown headings in `layout.md` may use emoji for doc navigation; **user-facing terminal strings** follow the semantic glyph table only.
-
-## Implementation
-
-- **Canonical helpers:** `lib/utils/cli-test-layout-chalk.js` (see symbol → helper table in [layout.md](./layout.md) § “Implementation map”).
-- **Preferred import alias:** `lib/utils/cli-layout-chalk.js` re-exports the same API when the path should not contain `test`.
-- **Prefer** `formatSuccessLine`, `formatSuccessParagraph`, `formatWarningLine`, `formatBlockingError`, `failureGlyph`, `successGlyph`, and other exported section helpers over ad hoc `` chalk.green(`✔ …`) `` for one-off success/error lines.
-- **Composite / indented lines** (e.g. `successGlyph()` + `chalk.white(…)`, or `  ✔ …` in validate-style output) may stay raw chalk **only** when a single helper would not match the spec; keep glyphs canonical.
-- **Tests:** extend or add tests under `tests/lib/utils/cli-test-layout-chalk.test.js` (and command tests) when behavior or snapshots encode layout.
-
-## Code entry points and maintenance
-
-- **Shared formatters:** `lib/utils/cli-test-layout-chalk.js` (canonical exports).
-- **Neutral re-export:** `lib/utils/cli-layout-chalk.js` → same module.
-- **Unit tests:** `tests/lib/utils/cli-test-layout-chalk.test.js`.
-
-**Optional follow-up:** If you touch centralized command error handling (e.g. `handleCommandError`), consider aligning it with `formatBlockingError` (but preserve `--json` / stdout-only behavior for script modes).
-
-## Commander and UX checklist
-
-- [ ] **Help**: `.description()` and `.option()` texts clear; default values and env overrides documented when behavior is non-obvious.
-- [ ] **Profile**: Matrix row added/updated for every new **leaf** command path.
-- [ ] **JSON / script**: If the command supports `--json` or machine output, JSON path skips TTY layout and stays stable for scripts.
-- [ ] **Errors**: Blocking failures use **✖** + red via shared helpers where possible; non-blocking issues use **⚠** + yellow.
-- [ ] **Docs**: User-facing CLI documentation lives under `docs/` per [docs-rules.mdc](./docs-rules.mdc) (command-centric; no raw HTTP/API tutorial in user docs unless that doc is explicitly contributor-focused).
-
-## Related plans
-
-Historical adoption context: `.cursor/plans/Done/129-cli_layout_adoption.plan.md`. Keep **cli-output-command-matrix.md** accurate when profiles or helper adoption change.

package/.cursor/rules/cli-product-neutral.mdc

@@ -1,43 +0,0 @@
----
-description: Apply when changing aifabrix CLI code, help text, errors, or user-facing docs under docs/commands. Keeps product output free of internal plans, training labs, and hardcoded integration names.
-alwaysApply: false
----
-
-# CLI product-neutral output (Builder)
-
-The **aifabrix** CLI is a **product**. User-visible strings must not encode internal engineering artifacts.
-
-## Forbidden in product paths
-
-Applies to **`lib/cli/**`**, **`lib/commands/**`**, CLI help (Commander `.description()` / `.option()` / `addHelpText`), **`lib/utils/*-error*.js`**, terminal messages from **`lib/utils/cli-*`**, and **`docs/commands/**`** (see [docs-rules.mdc](./docs-rules.mdc)).
-
-| Do not use | Use instead |
-| ---------- | ----------- |
-| Internal plan IDs (`Plan 331`, `plan 148`, `BK-1`) | User terms: *active datasource schema guards*, *schema replacement*, *overlap* |
-| Training / lab-only system keys baked into `if` branches (`sharepoint-e2e`, `hubspot-demo` in runtime logic) | **Dynamic** behavior from `systemKey`, `appName`, manifest, env.template, or secrets map |
-| Hardcoded vendor or connector names in generic helpers | Generic rules (path suffix, namespace, configuration keys from disk) |
-| Cursor plan file paths in errors or help | Link only in `.cursor/plans/` and contributor docs |
-
-**Examples in docs/commands** may name real apps (`hubspot-e2e`) when illustrating a command; **library code** must not special-case those names unless the user passed them as arguments.
-
-## Required patterns
-
-- **Missing `kv://` secrets:** Compare requested paths to the loaded secrets map generically (e.g. same path segment under a different namespace). Never hardcode `sharepoint` / `sharepoint-e2e` (or any other lab pair) in `lib/`.
-- **`--force` messaging:** Describe dataplane behavior (*bypass active-datasource schema replacement guards*), not internal plan numbers. Do **not** append extra CLI hints by matching English substrings on API errors — the dataplane pipeline already returns remediation in the 400 body when schema replacement is blocked.
-- **No `upload-pipeline-error-hints`-style modules:** Prefer forwarding `formatApiError` output; optional hints belong in the API contract, not duplicated heuristics in the CLI.
-- **Auth / pipeline errors:** Describe what the user can do (`aifabrix login`, `AIFABRIX_DEPLOYMENT_AUTH=client-credentials`), not ticket or plan IDs.
-- **JSDoc `@fileoverview`:** No plan numbers; optional brief behavior description only.
-
-## Verification
-
-Before merging CLI UX changes:
-
-1. `rg -i 'plan [0-9]|BK-[0-9]|sharepoint-e2e' lib/cli lib/commands lib/utils --glob '*.js'` → no matches in generic helpers (fixture strings in tests are OK).
-2. New error hints must work for **any** integration folder name the user owns.
-3. [cli-layout.mdc](./cli-layout.mdc) and [docs-rules.mdc](./docs-rules.mdc) still apply for glyphs and user-doc scope.
-
-## Related
-
-- [cli-layout.mdc](./cli-layout.mdc) — terminal layout and output profiles
-- [docs-rules.mdc](./docs-rules.mdc) — public CLI documentation audience
-- [project-rules.mdc](./project-rules.mdc) — ISO / security baseline