@tachyon-gg/railway-deploy 0.2.5 → 0.2.7

package/README.md

@@ -60,7 +60,7 @@ environment: production      # Railway environment name
 
 shared_variables:            # Variables shared across all services
   APP_ENV: production
-  DATABASE_URL: ${{Postgres.DATABASE_URL}}
+  API_PORT: "8080"
 
 services:                    # Map of service name -> config
   web: { ... }
@@ -212,6 +212,8 @@ variables:
 | `%{service_name}` | At config load time | Built-in: the service's config key |
 | `null` | N/A | Marks a variable for deletion |
 
+**Important:** Shared variables (`shared_variables`) cannot contain `${{service.VAR}}` references — Railway resolves shared variables without a service context, so cross-service references will resolve to empty strings. Use `${{service.VAR}}` references directly in service variables instead, and use shared variables only for plain values or `${{shared.OTHER_VAR}}` self-references.
+
 `%{param}` is expanded first, so it can be used inside `${{}}` Railway references. This is useful for templates that need to reference their own or other services' variables:
 
 ```yaml

package/dist/index.js

@@ -39225,11 +39225,19 @@ async function applyChangeset(client, changeset, projectId, environmentId, optio
   const failed = [];
   const noColor = options?.noColor ?? false;
   const createdServiceIds = new Map;
-  const varChangeIndices = changeset.changes.map((c, i) => c.type === "upsert-variables" || c.type === "upsert-shared-variables" ? i : -1).filter((i) => i >= 0);
-  const lastVarChangeIdx = varChangeIndices.length > 0 ? varChangeIndices[varChangeIndices.length - 1] : -1;
+  const lastVarChangeByService = new Map;
+  for (let i = 0;i < changeset.changes.length; i++) {
+    const c = changeset.changes[i];
+    if (c.type === "upsert-variables") {
+      lastVarChangeByService.set(c.serviceName, i);
+    }
+  }
   for (let i = 0;i < changeset.changes.length; i++) {
     const change = changeset.changes[i];
-    const skipDeploys = (change.type === "upsert-variables" || change.type === "upsert-shared-variables") && i < lastVarChangeIdx;
+    let skipDeploys = false;
+    if (change.type === "upsert-variables") {
+      skipDeploys = i < (lastVarChangeByService.get(change.serviceName) ?? -1);
+    }
     try {
       await applyChange(client, change, projectId, environmentId, createdServiceIds, skipDeploys);
       applied.push(change);
@@ -39645,8 +39653,10 @@ function diffServiceSettings(desired, current, changes) {
   if (!deepEqual(desired.source, current.source)) {
     settings.source = desired.source ?? null;
   }
-  if (desired.restartPolicy !== current.restartPolicy) {
-    settings.restartPolicy = desired.restartPolicy ?? null;
+  if (desired.restartPolicy !== undefined && desired.restartPolicy !== current.restartPolicy) {
+    settings.restartPolicy = desired.restartPolicy;
+  } else if (desired.restartPolicy === undefined && current.restartPolicy !== undefined) {
+    console.warn(`  Warning: "${desired.name}" has no restart_policy in config — Railway will keep "${current.restartPolicy}"`);
   }
   if (!deepEqual(desired.healthcheck, current.healthcheck)) {
     settings.healthcheck = desired.healthcheck ?? null;
@@ -39672,17 +39682,23 @@ function diffServiceSettings(desired, current, changes) {
   if (!deepEqual(desired.preDeployCommand, current.preDeployCommand)) {
     settings.preDeployCommand = desired.preDeployCommand ?? null;
   }
-  if (desired.restartPolicyMaxRetries !== current.restartPolicyMaxRetries) {
-    settings.restartPolicyMaxRetries = desired.restartPolicyMaxRetries ?? null;
+  if (desired.restartPolicyMaxRetries !== undefined && desired.restartPolicyMaxRetries !== current.restartPolicyMaxRetries) {
+    settings.restartPolicyMaxRetries = desired.restartPolicyMaxRetries;
+  } else if (desired.restartPolicyMaxRetries === undefined && current.restartPolicyMaxRetries !== undefined) {
+    console.warn(`  Warning: "${desired.name}" has no restart_policy_max_retries in config — Railway will keep ${current.restartPolicyMaxRetries}`);
   }
   if (desired.sleepApplication !== current.sleepApplication) {
     settings.sleepApplication = desired.sleepApplication ?? null;
   }
-  if (desired.builder !== current.builder) {
-    settings.builder = desired.builder ?? null;
+  if (desired.builder !== undefined && desired.builder !== current.builder) {
+    settings.builder = desired.builder;
+  } else if (desired.builder === undefined && current.builder !== undefined) {
+    console.warn(`  Warning: "${desired.name}" has no builder in config — Railway will keep "${current.builder}"`);
   }
-  if (!deepEqual(desired.watchPatterns, current.watchPatterns)) {
-    settings.watchPatterns = desired.watchPatterns ?? null;
+  if (desired.watchPatterns !== undefined && !deepEqual(desired.watchPatterns, current.watchPatterns)) {
+    settings.watchPatterns = desired.watchPatterns;
+  } else if (desired.watchPatterns === undefined && current.watchPatterns !== undefined) {
+    console.warn(`  Warning: "${desired.name}" has no watch_patterns in config — Railway will keep current patterns`);
   }
   if (desired.drainingSeconds !== current.drainingSeconds) {
     settings.drainingSeconds = desired.drainingSeconds ?? null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tachyon-gg/railway-deploy",
-  "version": "0.2.5",
+  "version": "0.2.7",
   "type": "module",
   "repository": {
     "type": "git",
@@ -29,7 +29,8 @@
     "test:all": "bun test --bail --timeout 30000 --env-file .env.test test/",
     "lint": "biome check .",
     "lint:fix": "biome check --write .",
-    "format": "biome format --write ."
+    "format": "biome format --write .",
+    "prepare": "husky"
   },
   "dependencies": {
     "@graphql-typed-document-node/core": "^3.2.0",
@@ -48,6 +49,7 @@
     "@graphql-codegen/typescript": "^4.1.2",
     "@graphql-codegen/typescript-operations": "^4.4.0",
     "@types/bun": "latest",
+    "husky": "^9.1.7",
     "typescript": "^5.7.0"
   }
 }