bmad-method-test-architecture-enterprise 2.0.1 → 2.0.2

package/.claude-plugin/marketplace.json

@@ -31,7 +31,7 @@
       "name": "bmad-method-test-architecture-enterprise",
       "source": "./",
       "description": "Master Test Architect module for quality strategy, test generation in red and expand modes with automatic run-and-heal, combined framework and CI setup, CI/CD quality gates, and structured testing education. The CI and ATDD commands remain available with existing customizations. Part of the BMad Method ecosystem.",
-      "version": "2.0.1",
+      "version": "2.0.2",
       "author": {
         "name": "Murat K Ozcan (TEA Creator) & Brian (BMad) Madison"
       },

package/README.md

@@ -59,16 +59,16 @@ Each skill can run directly in a fresh session.
 
 | Skill                                                                                                                           | Command                      | Menu | Use it to                                            |
 | ------------------------------------------------------------------------------------------------------------------------------- | ---------------------------- | ---- | ---------------------------------------------------- |
-| [Test Review](https://bmad-code-org.github.io/bmad-method-test-architecture-enterprise/how-to/workflows/run-test-review/)       | `/bmad-testarch-test-review` | RV   | Audit test quality and score findings                |
 | [Automate](https://bmad-code-org.github.io/bmad-method-test-architecture-enterprise/how-to/workflows/run-automate/)             | `/bmad-testarch-automate`    | TA   | Generate red acceptance scaffolds or expand coverage |
+| [Test Review](https://bmad-code-org.github.io/bmad-method-test-architecture-enterprise/how-to/workflows/run-test-review/)       | `/bmad-testarch-test-review` | RV   | Audit test quality and score findings                |
 | [Test Design](https://bmad-code-org.github.io/bmad-method-test-architecture-enterprise/how-to/workflows/run-test-design/)       | `/bmad-testarch-test-design` | TD   | Plan risks, coverage, and NFR evidence               |
 | [Framework](https://bmad-code-org.github.io/bmad-method-test-architecture-enterprise/how-to/workflows/setup-test-framework/)    | `/bmad-testarch-framework`   | TF   | Set up a test framework, CI, or both                 |
 | [NFR](https://bmad-code-org.github.io/bmad-method-test-architecture-enterprise/how-to/workflows/run-nfr-assess/)                | `/bmad-testarch-nfr`         | NR   | Audit implemented NFR evidence                       |
+| [Trace](https://bmad-code-org.github.io/bmad-method-test-architecture-enterprise/how-to/workflows/run-trace/)                   | `/bmad-testarch-trace`       | TR   | Map requirements to tests and decide a release gate  |
 | [Teach Me Testing](https://bmad-code-org.github.io/bmad-method-test-architecture-enterprise/how-to/workflows/teach-me-testing/) | `/bmad-teach-me-testing`     | TMT  | Learn testing through seven sessions                 |
 
-Additional skills:
+Evaluate:
 
-- [Trace](https://bmad-code-org.github.io/bmad-method-test-architecture-enterprise/how-to/workflows/run-trace/): `/bmad-testarch-trace`, menu `TR`. Map requirements to tests and decide a release gate.
 - [Evaluate](https://bmad-code-org.github.io/bmad-method-test-architecture-enterprise/tutorials/evaluate-your-first-skill/): `/bmad-testarch-evaluate`, menu `EV`. Build and run a behavioral evaluation.
 
 Framework and CI setup share one skill, `bmad-testarch-framework`.
@@ -114,6 +114,7 @@ npm install --save-dev bmad-method-test-architecture-enterprise
 ```
 
 - [`tea-test-review`](https://bmad-code-org.github.io/bmad-method-test-architecture-enterprise/reference/tea-test-review-cli/) runs the review skill against changed tests and returns a gate verdict.
+- [`tea-trace`](https://bmad-code-org.github.io/bmad-method-test-architecture-enterprise/reference/tea-trace-cli/) maps requirements to existing evidence and returns a coverage gate.
 - [`tea-evaluate`](https://bmad-code-org.github.io/bmad-method-test-architecture-enterprise/reference/tea-evaluate-cli/) checks, runs, scores, and compares behavioral evaluations.
 
 For `tea-evaluate`, install `eval-quality` alongside TEA in the evaluations folder:

package/cli/lib/build-prompt.js

@@ -291,9 +291,12 @@ function buildPrompt({
     'fragment set, Pact MCP) instead of inferring the flags.',
     'The two *_installed values above were read from the project manifest by the CLI. Do not re-derive them, and do',
     'not open package.json for them: they are the second half of each mandate gate, stated here for the same reason the flags are.',
-    'One read is allowed beyond the review set and the context list: when H10 turns on whether a static type checker',
-    '(mypy, pyright, tsc) runs in CI, read the CI workflow files (.github/workflows) and the scripts they invoke',
-    '(package.json scripts, a Makefile) for that one question. The row cannot be judged without it.',
+    'One evidence check is allowed beyond the review set and the context list: when H10 turns on whether a static',
+    'type checker (mypy, pyright, tsc) checks a type-assignability test, read the CI workflow files',
+    '(.github/workflows), scripts they invoke (package.json scripts, a Makefile), and the checker configuration',
+    '(such as pyproject.toml, mypy.ini, pyrightconfig.json, or tsconfig.json) for that question. Resolve the',
+    "checker's effective file scope: a CI command without file targets may use the config's files or include list.",
+    'H10 fires if the checker never reads the reviewed test file, even when CI runs the checker.',
     'tea_execution_mode and tea_capability_probe are the orchestration pair step-03-quality-evaluation.md branches on,',
     'resolved the same way the fragment keys above are: an explicit CLI flag, then [modules.tea] in the project config,',
     'then the module default. Both are stated because step-03 reads both, and "auto" with probing off resolves to',
@@ -344,7 +347,8 @@ function buildPrompt({
     JSON.stringify(contextFiles, null, 2),
     '---END CONTEXT---',
     '',
-    'Read only the artifacts named above. Never go looking for a story, PRD, or test design that the context list did',
+    'Read only the artifacts named above, except for the H10 checker evidence allowed earlier.',
+    'Never go looking for a story, PRD, or test design that the context list did',
     'not name: with no human present to confirm what you found, an unrequested artifact is a nondeterministic input.',
     '',
     'Context may RAISE a finding — a test that contradicts its acceptance criteria, a changed code path no assertion',

package/cli/lib/resolve-tea-config.js

@@ -56,11 +56,6 @@ const SNAPSHOT_DEFAULTS = {
   test_framework: 'auto',
 };
 
-const WORKFLOW_CUSTOM_PATHS = [
-  path.join('_bmad', 'custom', 'bmad-testarch-test-review.toml'),
-  path.join('_bmad', 'custom', 'bmad-testarch-test-review.user.toml'),
-];
-
 const PACT_MCP_VALUES = ['mcp', 'none'];
 // step-03-quality-evaluation.md's requestable modes. `auto` asks the capability
 // probe to pick; the other three are honoured as stated.
@@ -285,7 +280,7 @@ function parseToml(content, label) {
 }
 
 /** Resolve trusted skill defaults and project workflow overrides from the same configuration source. */
-function readWorkflowCustomization(source, skillRoot, projectRoot) {
+function readWorkflowCustomization(source, skillRoot, projectRoot, skillName = 'bmad-testarch-test-review') {
   if (!skillRoot) return {};
   const defaultsPath = path.join(skillRoot, 'customize.toml');
   let merged = {};
@@ -294,7 +289,9 @@ function readWorkflowCustomization(source, skillRoot, projectRoot) {
   } catch (error) {
     if (error.code !== 'ENOENT') throw configError(`Failed to load ${defaultsPath}: ${error.message}`);
   }
-  for (const relativePath of WORKFLOW_CUSTOM_PATHS) {
+  if (!/^[a-z0-9][a-z0-9-]*$/.test(skillName)) throw configError('Workflow skill name must be a lowercase kebab-case name');
+  const workflowPaths = ['.toml', '.user.toml'].map((suffix) => path.join('_bmad', 'custom', skillName + suffix));
+  for (const relativePath of workflowPaths) {
     const content = source.read(relativePath);
     if (content !== null) merged = mergeTables(merged, parseToml(content, source.label(relativePath)));
   }
@@ -497,10 +494,10 @@ const KEY_TO_INSTALLED_FIELD = {
  *   manifest rather than left to the agent.
  * @throws {Error} With code TEA_CONFIG_INVALID on unusable config content.
  */
-function resolveTeaConfig({ projectRoot, flags = {}, baseRef, skillRoot }) {
+function resolveTeaConfig({ projectRoot, flags = {}, baseRef, skillRoot, skillName }) {
   const source = configSource(projectRoot, baseRef);
   const file = readTeaConfigFile(projectRoot, baseRef, source);
-  const workflowCustomization = readWorkflowCustomization(source, skillRoot, projectRoot);
+  const workflowCustomization = readWorkflowCustomization(source, skillRoot, projectRoot, skillName);
 
   const values = {};
   const sources = {};

package/cli/lib/run-agent.js

@@ -38,6 +38,7 @@ const path = require('node:path');
 const { spawn, spawnSync } = require('node:child_process');
 const { AGENT_ADAPTERS, DEFAULT_CAPABILITIES, RUNNER_CAPABILITIES, bridgedArgsRefused, resolveModel } = require('./agent-adapters');
 const { launchPrefix } = require('./isolation-primitives');
+const { neutralizeWorkflowCommands } = require('./safe-log');
 
 const STDERR_TAIL_LINES = 20;
 const DEFAULT_TIMEOUT_MS = 1_800_000; // 30 minutes
@@ -280,7 +281,7 @@ function agentAnswer({ outcome, stdout, stderr }, { command: spawned, agentComma
   if (outcome.status !== 0) {
     const stderrTail = (stderr || '').trim().split('\n').slice(-STDERR_TAIL_LINES).join('\n');
     if (stderrTail) {
-      console.error(`Agent stderr (last ${STDERR_TAIL_LINES} lines):\n${stderrTail}`);
+      console.error(`Agent stderr (last ${STDERR_TAIL_LINES} lines):\n${neutralizeWorkflowCommands(stderrTail)}`);
     }
     // A signal-killed child (OOM, an external kill, anything outside the
     // ETIMEDOUT case already handled above) reports status as null; naming

package/cli/lib/safe-log.js

@@ -0,0 +1,7 @@
+/** Keep captured agent text from becoming a GitHub Actions workflow command. */
+function neutralizeWorkflowCommands(value) {
+  // The runner recognizes legacy ##[ commands anywhere in a log line.
+  return String(value).replaceAll('##[', '# #[').replaceAll('::', ': :');
+}
+
+module.exports = { neutralizeWorkflowCommands };