@tdk-landscape/tdk-cli-core 1.3.148 → 1.3.150

package/CHANGELOG.md

@@ -2,8 +2,9 @@
 
 Notable changes to the `tdk` CLI. Versions match [npm](https://www.npmjs.com/package/@tdk-landscape/tdk-cli-core?activeTab=versions) and the [binary releases](https://github.com/tdk-landscape/tdk-cli-releases/releases).
 
-## Unreleased
+## 1.3.131 (2026-10-12)
 
+- `healthCheckPath` is optional for `appType: "bring-your-own"` in `service.json`, so a service that is not HTTP (Postgres, Redis, MinIO, an SMTP relay) no longer needs a made-up `"/"` to validate. Without it TDK sends no HTTP probe to the service and gives its Traefik route no health check; the container counts as healthy once Docker reports it running. `port` stays required, and a set `healthCheckPath` behaves as before. Fixes #1086.
 - `tdk up` now publishes Traefik on host port 80 (and 443) when they are free, so routed URLs read `http://app.<project>.localhost/<app>` instead of `...localhost:8080/<app>`. It always wanted 80 but only ever tried 8080-8180. On macOS the free-port check treated the kernel's refusal of a non-root bind below 1024 as "busy", although Docker Desktop publishes those ports itself; that also made `TDK_HTTP_PORT=80` fail with "already in use". A project whose saved plan is on 8080 moves to 80 on its next `tdk up` once 80 is free. On Linux and WSL2 a non-root user cannot check port 80, so the 8080-8180 range is kept there. Printed URLs no longer end in `:80`.
 - TDK MCP tools now accept an absolute `projectPath` so a projectless MCP client can run doctor, inspect services and logs, and manage the intended project without depending on the server launch directory.
 - Shared platform stacks (postgres, traefik, proxy, messaging, infisical, monitoring, cdc, elk) now run in Compose projects named `<project>-<stack>` (for example `tdk-erp-system-database-management`) instead of the directory-named `database-management`, `tdk-out` and so on. Two TDK projects on one Docker daemon no longer share them.

package/cli/dist/commands/doctor.js

@@ -137,17 +137,25 @@ function isTimeout(err) {
     return isExecTimeout(err);
 }
 function createExecCheck(name, command, successMessage, failureMessage, fixInstructions) {
-    return async () => {
+    return async (exec = execAsync) => {
         try {
-            await execAsync(command, EXEC_TIMEOUT_MS);
+            await exec(command, EXEC_TIMEOUT_MS);
             return {
                 name,
                 didPass: true,
                 message: successMessage,
             };
         }
-        catch {
-            // Error details not needed - failure message tells user what to fix
+        catch (err) {
+            // A timeout means the command exists but did not answer in time, which is not the same as missing.
+            if (isTimeout(err)) {
+                return {
+                    name,
+                    didPass: false,
+                    message: `${name} did not respond within ${EXEC_TIMEOUT_MS / 1000}s`,
+                    fix: fixInstructions,
+                };
+            }
             return {
                 name,
                 didPass: false,

package/cli/dist/commands/import.js

@@ -3,10 +3,11 @@
 import { spawn } from "node:child_process";
 import { Command } from "commander";
 /**
- * The importer lives in its own repo so detectors can ship without a CLI release. Until it is
- * published to npm it is run from git; `TDK_IMPORT_PACKAGE` overrides the package spec.
+ * The importer lives in its own repo so it can ship without a CLI release, and is run from npm at a
+ * pinned version so a push to its main branch cannot change what `tdk import` does. Move the pin
+ * when a new importer is published. `TDK_IMPORT_PACKAGE` overrides the package spec.
  */
-export const IMPORT_PACKAGE = "github:tdk-landscape/tdk-import";
+export const IMPORT_PACKAGE = "@tdk-landscape/tdk-import@0.2.1";
 export function importInvocation(args, env = process.env, platform = process.platform) {
     const pkg = env.TDK_IMPORT_PACKAGE || IMPORT_PACKAGE;
     return {
@@ -31,8 +32,8 @@ export function runImport(args, spawnFn = spawn) {
     });
 }
 export const importCommand = new Command("import")
-    .description("Import the services a directory describes (runs tdk-import)")
-    .argument("[args...]", "Passed to tdk-import: [dir] --dry-run --yes --force --only <ids>")
+    .description("Import the services in a directory's root docker-compose.yml (runs tdk-import)")
+    .argument("[args...]", "Passed to tdk-import: [dir] --dry-run --yes --force")
     .allowUnknownOption()
     .helpOption(false)
     .action(async (args) => {