code-foundry 0.34.2 → 0.34.4

package/.github/workflows/release.yml

@@ -286,7 +286,11 @@ jobs:
   reconcile:
     name: Release / Reconcile
     needs: release
-    if: needs.release.result == 'success'
+    # A normal promotion push can successfully run Release Please without
+    # creating a release. Reconciliation is only meaningful after a release
+    # commit/tag is actually created; otherwise the promotion's source changes
+    # are incorrectly rejected as non-metadata commits.
+    if: needs.release.result == 'success' && needs.release.outputs.release_created == 'true'
     runs-on: ${{ inputs.runner }}
     timeout-minutes: 15
     permissions:

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## [0.34.4](https://github.com/0xPlayerOne/code-foundry/compare/v0.34.3...v0.34.4) (2026-08-02)
+
+
+### Bug Fixes
+
+* split native test suites by category ([93c9ede](https://github.com/0xPlayerOne/code-foundry/commit/93c9ede6e202a790befe1ee0e57633e3d01dc428))
+
+## [0.34.3](https://github.com/0xPlayerOne/code-foundry/compare/v0.34.2...v0.34.3) (2026-08-02)
+
+
+### Bug Fixes
+
+* **release:** reconcile only after creating a release ([#317](https://github.com/0xPlayerOne/code-foundry/issues/317)) ([#318](https://github.com/0xPlayerOne/code-foundry/issues/318)) ([6f2b00c](https://github.com/0xPlayerOne/code-foundry/commit/6f2b00c5f3fe27e56b448671902bf6f5e33cca1f))
+
 ## [0.34.2](https://github.com/0xPlayerOne/code-foundry/compare/v0.34.1...v0.34.2) (2026-08-01)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "code-foundry",
-  "version": "0.34.2",
+  "version": "0.34.4",
   "description": "A fast, language-aware repository factory for agent-ready workflows, testing, security, and release automation.",
   "type": "module",
   "license": "AGPL-3.0-or-later",
@@ -44,6 +44,7 @@
     "format:check": "node --check src/cli.mjs",
     "lint": "node --check src/cli.mjs",
     "test": "node --test",
+    "test:unit": "node --test",
     "type-check": "tsc --project jsconfig.json",
     "pack:check": "npm pack --dry-run",
     "prepublishOnly": "npm test && npm run pack:check"
@@ -52,7 +53,7 @@
     "access": "public"
   },
   "devDependencies": {
-    "@types/node": "^24.0.0",
-    "typescript": "^5.8.3"
+    "@types/node": "^26.0.0",
+    "typescript": "^7.0.2"
   }
 }

package/src/lib/test-discovery.mjs

@@ -0,0 +1,39 @@
+// @ts-check
+
+const TEST_FILE_PATTERN = /(?:\.(?:test|spec|unit|integration|smoke|e2e)\.(?:[cm]?[jt]sx?|py)|(?:^|\/)test_[^/]+\.py$|(?:^|\/)tests\/.*\.rs$)/i
+
+/** @param {string} file */
+export function isTestFile(file) {
+  return TEST_FILE_PATTERN.test(file.replaceAll('\\', '/'))
+}
+
+/** @param {string} file */
+export function testCategory(file) {
+  const normalized = file.replaceAll('\\', '/').toLowerCase()
+  const base = normalized.split('/').at(-1) ?? normalized
+
+  if (/(^|\/)(e2e|end-to-end|acceptance)(\/|$)|(?:^|[._-])(e2e|acceptance)(?:[._-]|$)/.test(normalized)) {
+    return 'e2e'
+  }
+  if (/(^|\/)(integration)(\/|$)|(?:^|[._-])integration(?:[._-]|$)/.test(normalized)) {
+    return 'integration'
+  }
+  if (/(^|\/)(smoke|live)(\/|$)|(?:^|[._-])(smoke|live)(?:[._-]|$)/.test(normalized)) {
+    return 'smoke'
+  }
+
+  if (normalized.endsWith('.rs') && /(^|\/)tests\//.test(normalized)) {
+    return 'integration'
+  }
+
+  // Rust files under tests/ compile as integration-test binaries. An explicit
+  // suffix still lets smoke and integration names win when they are present.
+  if (base.endsWith('_smoke.rs')) return 'smoke'
+  if (base.endsWith('_integration.rs')) return 'integration'
+  return 'unit'
+}
+
+/** @param {string[]} files @param {'unit'|'integration'|'e2e'|'smoke'} task */
+export function classifyTestFiles(files, task) {
+  return files.filter((file) => isTestFile(file) && testCategory(file) === task)
+}

package/src/runtime.mjs

@@ -6,6 +6,7 @@ import { resolve } from 'node:path'
 import { spawnSync } from 'node:child_process'
 import { detectPackageManager, resolveProfile } from './lib/profile.mjs'
 import { configured, readConfig } from './lib/config.mjs'
+import { classifyTestFiles } from './lib/test-discovery.mjs'
 import { classifyValidationMode, evaluateValidationGate } from './lib/validation-policy.mjs'
 import { readReleaseConfig, validateGeneratedReleaseDiff } from './lib/release-policy.mjs'
 
@@ -126,6 +127,19 @@ function capture(command, args = []) {
   return result.status === 0 ? result.stdout.trim() : ''
 }
 
+/** @returns {string[]} */
+function repositoryTestFiles() {
+  return capture('git', ['ls-files'])
+    .split(/\r?\n/)
+    .map((file) => file.trim())
+    .filter(Boolean)
+}
+
+/** @param {'unit'|'integration'|'e2e'|'smoke'} task */
+function taskTestFiles(task) {
+  return classifyTestFiles(repositoryTestFiles(), task)
+}
+
 /** @param {string[]} args @returns {[string|null, string[]]} */
 function packageCommand(args) {
   switch (packageManager) {
@@ -212,8 +226,16 @@ function relevant(task) {
     return (js && hasRootJavascriptProject()) || (python && hasRootPythonProject()) || (rust && hasRootRustProject())
   }
   if (['unit', 'integration', 'e2e', 'smoke'].includes(task)) {
-    return (js && hasRootJavascriptProject()) || (python && hasRootPythonProject()) ||
-      (rust && hasRootRustProject()) || (hasLanguage('solidity') && hasRootJavascriptProject())
+    if (taskTestFiles(/** @type {'unit'|'integration'|'e2e'|'smoke'} */ (task)).length > 0) return true
+    // A project with inline Rust/Python/JavaScript unit tests is still a unit
+    // test target even when it has no conventional test file path. Other
+    // categories must have an explicit script or discoverable test files so
+    // their workflow jobs become skipped instead of successful no-ops.
+    if (task === 'unit') {
+      return (js && hasRootJavascriptProject()) || (python && hasRootPythonProject()) ||
+        (rust && hasRootRustProject()) || (hasLanguage('solidity') && hasRootJavascriptProject())
+    }
+    return false
   }
   return true
 }
@@ -274,17 +296,49 @@ function ci(task) {
     smoke: ['test:smoke', 'smoke'],
   }
   const scripts = scriptsByTask[task]
-  if (scripts) runScript(scripts)
-  if (hasLanguage('rust') && hasRootRustProject()) run('cargo', ['test'])
+  const scripted = scripts ? runScript(scripts) : false
+  const testFiles = taskTestFiles(/** @type {'unit'|'integration'|'e2e'|'smoke'} */ (task))
+  const javascriptTests = testFiles.filter((file) => /\.(?:[cm]?[jt]sx?)$/i.test(file))
+  const pythonTests = testFiles.filter((file) => /\.py$/i.test(file))
+  const rustTests = testFiles.filter((file) => /\.rs$/i.test(file))
+
+  // A repository script is authoritative for that package. When it is absent,
+  // use Bun's native runner with only the discovered files for the requested
+  // category; this prevents smoke/integration files from being re-run as unit
+  // tests and avoids no-op category jobs.
+  if (!scripted && (hasLanguage('typescript') || hasLanguage('javascript') || hasLanguage('solidity')) && hasRootJavascriptProject()) {
+    if (javascriptTests.length > 0 || task === 'unit') run('bun', ['test', ...javascriptTests])
+  }
+
   if (hasLanguage('python') && hasRootPythonProject()) {
     const python = existsSync(resolve(root, '.venv/bin/python')) ? resolve(root, '.venv/bin/python') : 'python'
-    const integrationTests = resolve(root, 'tests/integration')
-    if (task !== 'integration' || existsSync(integrationTests)) {
-      run(python, ['-m', 'pytest', ...(task === 'integration' ? ['tests/integration'] : [])])
+    if (pythonTests.length > 0 || task === 'unit') run(python, ['-m', 'pytest', ...pythonTests])
+  }
+
+  if (hasLanguage('rust') && hasRootRustProject()) {
+    if (task === 'unit') {
+      if (existsSync(resolve(root, 'src/lib.rs'))) run('cargo', ['test', '--lib'])
+      if (existsSync(resolve(root, 'src/main.rs'))) run('cargo', ['test', '--bin', packageName()])
+      if (!existsSync(resolve(root, 'src/lib.rs')) && !existsSync(resolve(root, 'src/main.rs'))) run('cargo', ['test'])
+    } else {
+      for (const file of rustTests) {
+        const match = file.match(/^tests\/(.+)\.rs$/)
+        if (match && !match[1].includes('/')) run('cargo', ['test', '--test', match[1]])
+      }
     }
   }
 }
 
+function packageName() {
+  const cargo = readFileSafe(resolve(root, 'Cargo.toml'))
+  return cargo?.match(/^name\s*=\s*"([^"]+)"/m)?.[1] ?? 'app'
+}
+
+/** @param {string} file */
+function readFileSafe(file) {
+  try { return readFileSync(file, 'utf8') } catch { return null }
+}
+
 /** @param {string} task @param {string} ecosystem */
 function security(task, ecosystem) {
   if (task === 'profile') {