@tdk-landscape/tdk-cli-core 1.3.34 → 1.3.36

package/engine/topologies/tilt/generators/vite/frontend.star

@@ -16,6 +16,7 @@ load('../../../../topologies/tilt/manifest/constants.star', 'GENERATED_CONFIG_FI
 
 
 load('./templates.star', 'TEMPLATE_VITE_FRONTEND', 'TEMPLATE_VITE_FRONTEND_BUILD')
+load('../../../platform/docker/networking/api_path_constants.star', 'get_frontend_base_path')
 load('./helpers.star', 
     'build_header', 
     'generate_frontend_internal_aliases', 
@@ -56,7 +57,7 @@ def generate_frontend(manifest, backend_manifest=None, write_fn=None):
     stack = manifest.get('stack', 'app')
     port = manifest.get('port', BASE_PORT_FRONTEND)
     hmr_port = port + 1000  # HMR port is always port + 1000
-    base_path = manifest.get('basePath', '/' + stack + 's')
+    base_path = get_frontend_base_path(manifest)
     vite_base_path = _normalize_vite_base_path(base_path)
     app_name = manifest.get('appName', 'unknown')
 

package/engine/topologies/tilt/generators/vite/helpers.star

@@ -164,7 +164,7 @@ def generate_backend_path_aliases(resource_path, manifest):
     # hand-edited features: ["ddd"] with no key at all is still blocked, but real
     # entitlement is checked client-side by hasDddLicense() in extension-fetch.ts
     # before the CLI ever writes "ddd" into a resource's features array.
-    ddd_enabled = 'ddd' in manifest.get('features', []) and os.environ.get('TDK_LICENSE_KEY', '') != ''
+    ddd_enabled = 'ddd' in manifest.get('featuresEnabled', []) and os.environ.get('TDK_LICENSE_KEY', '') != ''
 
     if ddd_enabled:
         ddd_aliases = [

package/engine/topologies/tilt/manifest/constants.star

@@ -155,7 +155,7 @@ MANIFEST_DEFAULTS = {
     'healthCheckPath': HEALTH_CHECK_PATH,
     'internalPort': BASE_PORT_FRONTEND,
     'replicas': 1,
-    'features': [],
+    'featuresEnabled': [],
     'envVars': {},
     'syncs': None,
     'dependsOn': [],

package/engine/topologies/tilt/manifest/mapper.star

@@ -31,7 +31,7 @@ FIELD_MAPPINGS = {
     'replicas': ['replicas', 'instances', 'count', 'spec.replicas', 'metadata.replicas'],
     
     # Feature flags
-    'features': ['features', 'capabilities', 'spec.features', 'enabledFeatures', 'app.features'],
+    'featuresEnabled': ['featuresEnabled', 'capabilities', 'spec.features', 'enabledFeatures', 'app.features'],
     
     # Dependencies
     'dependsOn': [
@@ -199,7 +199,7 @@ def _to_yaml_format(internal_manifest):
             'port': internal_manifest.get('port', 4000),
             'runtime': internal_manifest.get('runtime', 'bun'),
             'replicas': internal_manifest.get('replicas', 1),
-            'features': internal_manifest.get('features', []),
+            'featuresEnabled': internal_manifest.get('featuresEnabled', []),
             'envVars': internal_manifest.get('envVars', {}),
             'dependencies': {
                 'internal': internal_manifest.get('dependsOn', []),

package/engine/topologies/tilt/manifest/parser.star

@@ -90,7 +90,7 @@ def normalize(manifest, resource_path=""):
     # Copy all existing fields
     for key, value in manifest.items():
         normalized[key] = value
-    
+
     # 1. Apply defaults for missing fields
     for field, default_value in MANIFEST_DEFAULTS.items():
         if field not in normalized or normalized[field] == None:
@@ -111,7 +111,7 @@ def normalize(manifest, resource_path=""):
     if normalized.get('syncs') == None or normalized.get('syncs') == []:
         normalized['syncs'] = _compute_syncs(
             normalized['appType'],
-            normalized.get('features', [])
+            normalized.get('featuresEnabled', [])
         )
     
     # Compute backendName for frontends if missing

package/engine/topologies/tilt/manifest/schema.star

@@ -117,7 +117,7 @@ MANIFEST_SCHEMA = {
         'description': 'Runtime environment',
         'default': RUNTIME,
     },
-    'features': {
+    'featuresEnabled': {
         'type': 'list',
         'required': False,
         'constraints': {
@@ -263,7 +263,7 @@ CROSS_FIELD_CONSTRAINTS = {
         'message': 'Frontend resources must specify backendName and basePath',
     },
     'prisma_requires_database': {
-        'condition': {'field': 'features', 'contains': 'prisma'},
+        'condition': {'field': 'featuresEnabled', 'contains': 'prisma'},
         'requires': ['databaseName'],
         'severity': 'warning',
         'message': 'Resources using Prisma should specify databaseName',

package/engine/topologies/tilt/manifest/tests/test_loader_parser.star

@@ -281,7 +281,7 @@ def _test_manifest_merging():
         'appName': 'test-service',
         'appType': 'backend',
         'port': 4000,
-        'features': ['nats'],
+        'featuresEnabled': ['nats'],
         'envVars': {
             'SHARED_VAR': 'base_value',
         },
@@ -289,7 +289,7 @@ def _test_manifest_merging():
     
     overlay = {
         'port': 4001,  # Override
-        'features': ['prisma'],  # Override
+        'featuresEnabled': ['prisma'],  # Override
         'envVars': {
             'SHARED_VAR': 'overlay_value',  # Override
             'NEW_VAR': 'new_value',  # Add
@@ -300,7 +300,7 @@ def _test_manifest_merging():
     
     assert_equal('test-service', merged['appName'], "Should preserve base appName")
     assert_equal(4001, merged['port'], "Should overlay port")
-    assert_equal(['prisma'], merged['features'], "Should overlay features")
+    assert_equal(['prisma'], merged['featuresEnabled'], "Should overlay features")
     assert_equal('overlay_value', merged['envVars']['SHARED_VAR'], "Should overlay nested env var")
     assert_equal('new_value', merged['envVars']['NEW_VAR'], "Should add new env var")
     

package/engine/topologies/tilt/manifest/tests/test_validator.star

@@ -213,7 +213,7 @@ def _test_cross_field_validation():
         'appType': 'backend',
         'stack': 'user',
         'port': 4001,
-        'features': ['prisma'],
+        'featuresEnabled': ['prisma'],
         # Missing databaseName
     }
     
@@ -312,7 +312,7 @@ def _test_full_validation():
         'port': 4000,
         'replicas': 1,
         'runtime': 'bun',
-        'features': ['nats', 'prisma'],
+        'featuresEnabled': ['nats', 'prisma'],
         'databaseName': 'TDK_user',
         'dependsOn': [],
         'traefik': {

package/engine/topologies/tilt/manifest/validator.star

@@ -316,16 +316,16 @@ def _validate_values(manifest, context):
             stats['rules_passed'] += 1
     
     # Validate features length
-    if 'features' in manifest:
+    if 'featuresEnabled' in manifest:
         stats['fields_checked'] += 1
-        features = manifest['features']
-        
+        features = manifest.get('featuresEnabled', [])
+
         if len(features) > VALIDATION_THRESHOLDS['max_features']:
             warnings.append(ManifestErrors.new(
                 message="Too many features ({}), consider splitting resource".format(len(features)),
                 category=ManifestErrors.CATEGORY['VALIDATION'],
                 severity=ManifestErrors.SEVERITY['WARNING'],
-                context={'field': 'features', 'count': len(features), 'max': VALIDATION_THRESHOLDS['max_features']},
+                context={'field': 'featuresEnabled', 'count': len(features), 'max': VALIDATION_THRESHOLDS['max_features']},
             ))
         else:
             stats['rules_passed'] += 1
@@ -374,16 +374,16 @@ def _validate_cross_field(manifest, context):
             stats['rules_passed'] += 1
     
     # Prisma requires databaseName
-    features = manifest.get('features', [])
+    features = manifest.get('featuresEnabled', [])
     if 'prisma' in features:
         stats['fields_checked'] += 1
-        
+
         if not manifest.get('databaseName'):
             warnings.append(ManifestErrors.new(
                 message="Resource with 'prisma' feature should specify databaseName",
                 category=ManifestErrors.CATEGORY['VALIDATION'],
                 severity=ManifestErrors.SEVERITY['WARNING'],
-                context={'field': 'databaseName', 'features': features},
+                context={'field': 'databaseName', 'featuresEnabled': features},
             ))
         else:
             stats['rules_passed'] += 1

package/engine/topologies/tilt/resources/orchestrator/generators/env.star

@@ -14,6 +14,7 @@ OUTPUT = struct(AUTOGENERATED_FOLDER = ".autogenerated")
 
 load("../../../../platform/docker/auth/auth.star", "AuthConfig")
 load("../../../../platform/docker/networking/traefik_helpers.star", "get_api_path")
+load("../../../../platform/docker/networking/api_path_constants.star", "get_frontend_base_path")
 load("../../../../platform/docker/constants.star", "PlatformDockerConstants")
 
 
@@ -133,8 +134,8 @@ def _generate_params_env(manifest, backend_manifest=None, write_fn=None):
         backend_name = manifest.get('backendName', stack + '-backend')
         target_filename = '/' + OUTPUT.AUTOGENERATED_FOLDER + '/.env.frontend.autogenerated'
         
-        # Get basePath from manifest or default to /{stack}
-        base_path = manifest.get('basePath', '/' + stack)
+        # Shared with the Traefik route and the Vite `base` -- see get_frontend_base_path.
+        base_path = get_frontend_base_path(manifest)
         public_url = 'http://' + PlatformDockerConstants.LOCAL_DOMAIN + base_path
         
         content = """# Generated by Tilt SDK - DO NOT EDIT

package/engine/topologies/tilt/resources/orchestrator/generators/manifest_resource.star

@@ -360,7 +360,8 @@ def _generate_all_configs_for_resource(
         dockerfile_content = Docker.backend(
             res_path=resource_path,
             resource_name=resource_name,
-            use_golden=should_enable('golden-image')
+            use_golden=should_enable('golden-image'),
+            use_prisma='prisma' in manifest.get('featuresEnabled', [])
         )
     
     write_file(resource_path + "/" + OUTPUT.AUTOGENERATED_FOLDER + "/" + GENERATED_CONFIG_FILENAMES['dockerfile_app'], dockerfile_content)
@@ -404,7 +405,7 @@ def _generate_all_configs_for_resource(
         TSConfig.frontend(resource_path, write_file, internal_deps=internal_deps_map, is_docker=False)
     else:
         prisma_path = resource_config.get('prisma_client_path', './node_modules/.prisma/client')
-        resource_features = manifest.get('features', [])
+        resource_features = manifest.get('featuresEnabled', [])
         TSConfig.backend(resource_path, prisma_path, write_file, internal_deps=internal_deps_map, is_docker=True, features=resource_features)
         TSConfig.backend(resource_path, prisma_path, write_file, internal_deps=internal_deps_map, is_docker=False, features=resource_features)
         

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tdk-landscape/tdk-cli-core",
-  "version": "1.3.34",
+  "version": "1.3.36",
   "description": "TDK CLI - Tilt Development Kit command line interface",
   "keywords": [
     "tdk",

package/engine/schemas/manifest-schema.json

@@ -1,301 +0,0 @@
-{
-  "$id": "https://TDK Landscape.local/schemas/service-schema.json",
-  "$schema": "http://json-schema.org/draft-07/schema#",
-  "additionalProperties": false,
-  "allOf": [
-    {
-      "if": {
-        "properties": {
-          "appType": {
-            "const": "backend"
-          }
-        }
-      },
-      "then": {
-        "properties": {
-          "port": {
-            "maximum": 5999,
-            "minimum": 4000
-          }
-        },
-        "required": [
-          "port"
-        ]
-      }
-    },
-    {
-      "if": {
-        "properties": {
-          "appType": {
-            "const": "frontend"
-          }
-        }
-      },
-      "then": {
-        "properties": {
-          "port": {
-            "maximum": 3999,
-            "minimum": 3000
-          }
-        }
-      }
-    },
-    {
-      "if": {
-        "properties": {
-          "appType": {
-            "const": "worker"
-          }
-        }
-      },
-      "then": {
-        "properties": {
-          "features": {
-            "contains": {
-              "const": "nats"
-            }
-          }
-        },
-        "required": [
-          "features"
-        ]
-      }
-    }
-  ],
-  "description": "Configuration for TDK Landscape services. Drives Tilt orchestration, Docker Compose generation, and dependency resolution. Supports both explicit service.json files and synthesis from directory structure (CEO Review: rename-service-manifest).",
-  "properties": {
-    "$schema": {
-      "description": "JSON Schema reference for IDE validation",
-      "type": "string"
-    },
-    "apiBasePath": {
-      "default": "/api",
-      "description": "Base path for API routes in Traefik.",
-      "pattern": "^/[a-z0-9/-]*$",
-      "type": "string"
-    },
-    "appName": {
-      "description": "Unique resource identifier. Must match directory name.\nFormat: {stack}-{function}-{type}\nExample: 'user-management-backend'",
-      "examples": [
-        "user-management-backend",
-        "order-planner-frontend"
-      ],
-      "maxLength": 64,
-      "minLength": 3,
-      "pattern": "^[a-z][a-z0-9-]*[a-z0-9]$",
-      "type": "string"
-    },
-    "appType": {
-      "description": "Service type determining build strategy:\n• backend - Hono/Express API server\n• frontend - Vite/React application\n• sdk - Shared library package\n• worker - Background job processor\n• migrator - Database migration runner\n• library - Shared code library",
-      "enum": [
-        "backend",
-        "frontend",
-        "sdk",
-        "worker",
-        "migrator",
-        "library"
-      ],
-      "type": "string"
-    },
-    "backendName": {
-      "description": "For frontends: associated backend service.\nAuto-computed from appName if not specified.",
-      "pattern": "^[a-z][a-z0-9-]*-backend$",
-      "type": "string"
-    },
-    "basePath": {
-      "description": "Base path for frontend routing on TDK Landscape.localhost. Default is /{stack}s when omitted.",
-      "pattern": "^/[a-z0-9/-]*$",
-      "type": "string"
-    },
-    "databaseName": {
-      "description": "Override auto-generated DB name.\nDefault: TDK_{stack}",
-      "pattern": "^TDK_[a-z_]+$",
-      "type": "string"
-    },
-    "dependencies": {
-      "description": "Deprecated. Use 'dependsOn' instead.",
-      "deprecated": true,
-      "items": {
-        "pattern": "^[a-z][a-z0-9-]*$",
-        "type": "string"
-      },
-      "type": "array",
-      "uniqueItems": true
-    },
-    "dockerfile": {
-      "default": "autogenerated/Dockerfile.app.autogenerated",
-      "description": "Custom Dockerfile path relative to service directory.",
-      "type": "string"
-    },
-    "stack": {
-      "description": "Technology stack this resource belongs to.\nUsed for:\n• Database naming: TDK_{stack}\n• Traefik routing: /api/{stack}\n• NATS queue: {stack}_backend_svc",
-      "enum": [
-        "user",
-        "team",
-        "auth",
-        "order",
-        "order-planner",
-        "service",
-        "inventory",
-        "billing",
-        "notification",
-        "analytics",
-        "profile",
-        "platform",
-        "product"
-      ],
-      "type": "string"
-    },
-    "params": {
-      "additionalProperties": {
-        "type": "string"
-      },
-      "description": "Non-secret configuration parameters (BASE_URL, PORT, etc.). Secrets use 'secrets' section below.",
-      "type": "object"
-    },
-    "secrets": {
-      "description": "Secret references resolved via Infisical. NO secrets in params/envVars - security violation.",
-      "properties": {
-        "required": {
-          "description": "Required secret names. Service CRASHES if any are missing.",
-          "items": {
-            "type": "string",
-            "pattern": "^[A-Z_][A-Z0-9_]*$"
-          },
-          "type": "array",
-          "uniqueItems": true
-        },
-        "optional": {
-          "description": "Optional secret names. Service starts but warns if missing.",
-          "items": {
-            "type": "string",
-            "pattern": "^[A-Z_][A-Z0-9_]*$"
-          },
-          "type": "array",
-          "uniqueItems": true
-        }
-      },
-      "required": [
-        "required"
-      ],
-      "type": "object"
-    },
-    "features": {
-      "description": "Platform features:\n• nats - NATS JetStream eventing\n• prisma - DB ORM & migrations\n• redis - Caching layer\n• infisical - Secrets management\n• vitest - Test framework\n• traefik - Reverse proxy routing\n• websocket - WS support\n• graphql - GraphQL server\n• grpc - gRPC protocol",
-      "items": {
-        "enum": [
-          "nats",
-          "prisma",
-          "redis",
-          "infisical",
-          "vitest",
-          "traefik",
-          "websocket",
-          "graphql",
-          "grpc"
-        ],
-        "type": "string"
-      },
-      "type": "array",
-      "uniqueItems": true
-    },
-    "healthCheckPath": {
-      "default": "/health",
-      "description": "HTTP endpoint for health checks. Must return 200 OK.",
-      "pattern": "^/[a-z0-9/-]*$",
-      "type": "string"
-    },
-    "dependsOn": {
-      "description": "Resource dependencies used for startup ordering and frontend dependency env generation.\nEach entry can be a stack name (e.g. 'user') or resource name (e.g. 'user-management-backend').\nServices listed here start before this service. Used for focus mode, health checks, migration order.",
-      "items": {
-        "pattern": "^[a-z][a-z0-9-]*$",
-        "type": "string"
-      },
-      "type": "array",
-      "uniqueItems": true
-    },
-    "nats": {
-      "description": "Custom NATS configuration. Auto-generated if 'nats' in features.",
-      "properties": {
-        "queueGroup": {
-          "description": "NATS queue group for load balancing",
-          "type": "string"
-        },
-        "subjects": {
-          "description": "NATS subjects to subscribe",
-          "items": {
-            "type": "string"
-          },
-          "type": "array"
-        }
-      },
-      "type": "object"
-    },
-    "playwright": {
-      "default": false,
-      "description": "Generate Playwright config in .autogenerated/playwright.config.autogenerated.ts when true.",
-      "type": "boolean"
-    },
-    "port": {
-      "description": "External dev port (internal always 3000).\nAssignments:\n• 4000: user\n• 4001: service\n• 4002: order\n• 4003: team\n• 4004: auth",
-      "examples": [
-        4000,
-        4001,
-        4002,
-        4003,
-        4004
-      ],
-      "maximum": 9999,
-      "minimum": 3000,
-      "type": "integer"
-    },
-    "replicas": {
-      "default": 1,
-      "description": "Container instances to run.\nCreates separate containers (service-1, service-2).\nTraefik load-balances between all replicas.",
-      "maximum": 10,
-      "minimum": 1,
-      "type": "integer"
-    },
-    "runtime": {
-      "default": "bun",
-      "description": "Runtime to use for this service",
-      "enum": [
-        "node",
-        "bun"
-      ],
-      "type": "string"
-    },
-    "syncs": {
-      "description": "Directories to sync for hot-reload.\nAuto-computed if not specified:\n• backend: ['src', 'prisma']\n• frontend: ['src', 'public']",
-      "items": {
-        "type": "string"
-      },
-      "type": "array"
-    },
-    "traefik": {
-      "description": "Custom Traefik routing. Auto-generated if not specified.",
-      "properties": {
-        "healthCheck": {
-          "description": "Health check path for Traefik",
-          "type": "string"
-        },
-        "host": {
-          "description": "Custom hostname (e.g., 'user.backend.{project}.local')",
-          "type": "string"
-        },
-        "pathPrefix": {
-          "description": "URL path prefix (e.g., '/api/user')",
-          "type": "string"
-        }
-      },
-      "type": "object"
-    }
-  },
-  "required": [
-    "appName",
-    "appType",
-    "stack"
-  ],
-  "title": "TDK Landscape Service Configuration",
-  "type": "object"
-}