quiltdata
diff --git a/‎CHANGELOG.md‎
Lines changed: 15 additions & 0 deletions b/‎CHANGELOG.md‎
Lines changed: 15 additions & 0 deletions
diff --git a/‎bin/cli.ts‎
Lines changed: 9 additions & 9 deletions b/‎bin/cli.ts‎
Lines changed: 9 additions & 9 deletions
diff --git a/‎bin/commands/status.ts‎
Lines changed: 2 additions & 14 deletions b/‎bin/commands/status.ts‎
Lines changed: 2 additions & 14 deletions
diff --git a/‎docker/Makefile‎
Lines changed: 52 additions & 6 deletions b/‎docker/Makefile‎
Lines changed: 52 additions & 6 deletions
diff --git a/‎docker/app-manifest.yaml‎
Lines changed: 1 addition & 1 deletion b/‎docker/app-manifest.yaml‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎docker/pyproject.toml‎
Lines changed: 4 additions & 3 deletions b/‎docker/pyproject.toml‎
Lines changed: 4 additions & 3 deletions
diff --git a/‎docker/pyrightconfig.json‎
Lines changed: 40 additions & 0 deletions b/‎docker/pyrightconfig.json‎
Lines changed: 40 additions & 0 deletions
diff --git a/‎docker/scripts/docker.py‎
Lines changed: 4 additions & 3 deletions b/‎docker/scripts/docker.py‎
Lines changed: 4 additions & 3 deletions
diff --git a/‎docker/scripts/experiment_search_syntax.py‎
Lines changed: 1 addition & 1 deletion b/‎docker/scripts/experiment_search_syntax.py‎
Lines changed: 1 addition & 1 deletion
@@ -5,6 +5,21 @@ All notable changes to this project will be documented in this file.
 
 ## [Unreleased]
 
+### Added
+
+- Browse buttons for linked packages in Benchling Canvas
+
+### Changed
+
+- **BREAKING**: Removed `--config` alias, use only `--profile` for all commands
+
+### Fixed
+
+- **CRITICAL**: `--profile` flag now works correctly for all commands
+- Configuration validation allows additional properties for backward compatibility
+- Status command works with any profile containing stackArn (not just integrated stacks)
+- Python package license corrected to Apache-2.0
+
 ## [0.8.3] - 2025-11-18
 
 ### Fixed
 
@@ -60,7 +60,7 @@ program
     )
     .option("--env-file <path>", "Path to .env file", ".env")
 // Multi-environment options (v0.7.0)
-    .option("--profile, --config <name>", "Configuration profile to use (default: default)")
+    .option("--profile <name>", "Configuration profile to use (default: default)")
     .option("--stage <name>", "API Gateway stage: dev or prod (default: prod)")
 // Common options
     .option("--no-bootstrap-check", "Skip CDK bootstrap verification")
@@ -101,7 +101,7 @@ For more information: https://github.com/quiltdata/benchling-webhook#deployment
 program
     .command("setup")
     .description("Run setup wizard only (without deployment)")
-    .option("--profile, --config <name>", "Configuration profile to use (default: default)")
+    .option("--profile <name>", "Configuration profile to use (default: default)")
     .option("--inherit-from <name>", "Base profile to inherit from")
     .option("--region <region>", "AWS region")
     .option("--aws-profile <name>", "AWS credentials profile")
@@ -122,7 +122,7 @@ program
         "Check CloudFormation stack status and BenchlingIntegration parameter",
     )
     .option(
-        "--profile, --config <name>",
+        "--profile <name>",
         "Configuration profile to check (default: default)",
     )
     .option("--aws-profile <name>", "AWS credentials profile")
@@ -174,7 +174,7 @@ Use --no-exit to keep monitoring indefinitely.
 program
     .command("logs")
     .description("View CloudWatch logs from deployed webhook integration")
-    .option("--profile, --config <name>", "Configuration profile to use (default: default)")
+    .option("--profile <name>", "Configuration profile to use (default: default)")
     .option("--aws-profile <name>", "AWS credentials profile")
     .option(
         "--type <type>",
@@ -254,7 +254,7 @@ program
 program
     .command("validate")
     .description("Validate configuration without deploying")
-    .option("--profile, --config <name>", "Configuration profile to validate", "default")
+    .option("--profile <name>", "Configuration profile to validate", "default")
     .option("--verbose", "Show detailed validation information")
     .action(async (options) => {
         try {
@@ -300,7 +300,7 @@ program
     .command("manifest")
     .description("Generate Benchling app manifest file")
     .option("--output <path>", "Output file path", "app-manifest.yaml")
-    .option("--profile, --config <name>", "Configuration profile to use (optional)")
+    .option("--profile <name>", "Configuration profile to use (optional)")
     .option("--catalog <url>", "Catalog URL to use (optional, overrides profile)")
     .action(async (options) => {
         try {
@@ -362,7 +362,7 @@ For more information: https://github.com/quiltdata/benchling-webhook#multi-envir
 program
     .command("health-check")
     .description("Check configuration health and secrets sync status")
-    .option("--profile, --config <name>", "Configuration profile to check", "default")
+    .option("--profile <name>", "Configuration profile to check", "default")
     .option("--json", "Output in JSON format")
     .action(async (options) => {
         try {
@@ -377,7 +377,7 @@ program
 program
     .command("config")
     .description("Show configuration for a profile as JSON")
-    .option("--profile, --config-name <name>", "Configuration profile to show", "default")
+    .option("--profile <name>", "Configuration profile to show", "default")
     .action(async (options) => {
         try {
             await configShowCommand(options);
@@ -414,7 +414,7 @@ if (
             options.yes = true;
         } else if (args[i] === "--setup-only") {
             options.setupOnly = true;
-        } else if ((args[i] === "--profile" || args[i] === "--config") && i + 1 < args.length) {
+        } else if (args[i] === "--profile" && i + 1 < args.length) {
             options.profile = args[i + 1];
             i++;
         } else if (args[i] === "--inherit-from" && i + 1 < args.length) {
 
@@ -877,24 +877,12 @@ export async function statusCommand(options: StatusCommandOptions = {}): Promise
         };
     }
 
-    // Check if integrated stack
-    if (!config.integratedStack) {
-        const errorMsg = "Status command is only available for integrated stack mode";
-        console.log(chalk.yellow(`\n⚠️  ${errorMsg}\n`));
-        console.log(chalk.dim("This profile is configured for standalone deployment."));
-        console.log(chalk.dim("Use CloudFormation console to check webhook stack status.\n"));
-        return {
-            success: false,
-            error: errorMsg,
-        };
-    }
-
-    // Extract stack info
+    // Extract stack info - if stackArn is present, status should work regardless of integratedStack flag
     const stackArn = config.quilt.stackArn;
     if (!stackArn) {
         return {
             success: false,
-            error: "Quilt stack ARN not found in configuration. This command requires a Quilt stack ARN to check integration status.",
+            error: "Quilt stack ARN not found in configuration. This command requires a Quilt stack ARN to check stack status.",
         };
     }
     const region = config.deployment.region;
 
@@ -3,6 +3,10 @@
 # Configuration is now managed via XDG (~/.config/benchling-webhook/{profile}/)
 # with unified config.json and deployments.json per profile (v0.7.0)
 
+# Load .env file from project root if it exists (for NGROK_DOMAIN and other env vars)
+-include ../.env
+export
+
 # Include deployment targets
 include make.deploy
 
@@ -36,10 +40,10 @@ help:
 	@echo "Development Servers (all use xdg-launch):"
 	@echo "  run / run-dev         - Docker dev with hot-reload (port $(PORT_DOCKER_DEV))"
 	@echo "  run-prod              - Docker production mode (port $(PORT_DOCKER_PROD))"
-	@echo "  run-ngrok             - Docker dev + ngrok tunnel (port $(PORT_DOCKER_DEV))"
+	@echo "  run-ngrok             - Docker dev + ngrok tunnel (requires NGROK_DOMAIN env var)"
 	@echo "  run-native            - Native Flask with dev profile (port $(PORT_NATIVE))"
 	@echo "  run-native-verbose    - Alias for run-native (always verbose)"
-	@echo "  run-native-ngrok      - Native server + ngrok tunnel (port $(PORT_NATIVE))"
+	@echo "  run-native-ngrok      - Native server + ngrok tunnel (requires NGROK_DOMAIN env var)"
 	@echo "  run-ecr               - Pull and run image from ECR (port $(PORT_DOCKER_PROD))"
 	@echo ""
 	@echo "Testing (xdg-launch-based tests marked with *):"
@@ -55,7 +59,7 @@ help:
 	@echo "  test-benchling        - Test Benchling OAuth credentials"
 	@echo "  test-query            - Test Quilt package query (defaults to BENCHLING_TEST_ENTRY)"
 	@echo "  test-integration      - Full integration tests (requires real Benchling creds)"
-	@echo "  lint                  - Auto-fix code formatting"
+	@echo "  lint                  - Auto-fix code formatting (black + isort) and type check (pyright)"
 	@echo ""
 	@echo "Health & Monitoring:"
 	@echo "  health                - Check docker prod health (port $(PORT_DOCKER_PROD))"
@@ -135,6 +139,19 @@ run: check-xdg
 run-dev: run
 
 run-ngrok:
+	@if [ -z "$(NGROK_DOMAIN)" ]; then \
+		echo "❌ Error: NGROK_DOMAIN environment variable is required"; \
+		echo ""; \
+		echo "Get your static domain from: https://dashboard.ngrok.com/domains"; \
+		echo ""; \
+		echo "Usage:"; \
+		echo "  export NGROK_DOMAIN=uniformly-alive-halibut.ngrok-free.app"; \
+		echo "  make run-ngrok"; \
+		exit 1; \
+	fi
+	@echo "🚀 Starting dev server and ngrok tunnel..."
+	@echo "📡 Public URL: https://$(NGROK_DOMAIN)/"
+	@echo ""
 	@make run &
 	@make ngrok PORT=$(PORT_DOCKER_DEV) SLEEP=10
 
@@ -152,10 +169,22 @@ run-native-verbose: run-native
 
 # Native server + ngrok tunnel
 run-native-ngrok: check-xdg
-	@echo "Starting native server and ngrok tunnel..."
+	@if [ -z "$(NGROK_DOMAIN)" ]; then \
+		echo "❌ Error: NGROK_DOMAIN environment variable is required"; \
+		echo ""; \
+		echo "Get your static domain from: https://dashboard.ngrok.com/domains"; \
+		echo ""; \
+		echo "Usage:"; \
+		echo "  export NGROK_DOMAIN=uniformly-alive-halibut.ngrok-free.app"; \
+		echo "  make run-native-ngrok"; \
+		exit 1; \
+	fi
+	@echo "🚀 Starting native server and ngrok tunnel..."
+	@echo "📡 Public URL: https://$(NGROK_DOMAIN)/"
+	@echo ""
 	@echo "Press Ctrl+C to stop both ngrok and server"
 	@make run-native-verbose &
-	@make ngrok
+	@make ngrok PORT=$(PORT_NATIVE)
 
 # Run ECR image locally (assumes image exists in ECR)
 run-ecr: check-xdg docker-ecr-login
@@ -303,31 +332,48 @@ test-deployed-dev: check-xdg
 test-deployed-dev-direct: check-xdg
 	@echo "🧪 Testing deployed dev stack (profile: $(PROFILE))..."
 	@DEV_ENDPOINT=$$(jq -r '.active.dev.endpoint // empty' $(PROFILE_DIR)/deployments.json 2>/dev/null); \
+	IMAGE_TAG=$$(jq -r '.active.dev.imageTag // empty' $(PROFILE_DIR)/deployments.json 2>/dev/null); \
+	DEPLOYED_AT=$$(jq -r '.active.dev.timestamp // empty' $(PROFILE_DIR)/deployments.json 2>/dev/null); \
 	if [ -z "$$DEV_ENDPOINT" ]; then \
 		echo "❌ No dev endpoint found in $(PROFILE_DIR)/deployments.json"; \
 		echo "💡 Run 'npm run deploy:dev -- --profile $(PROFILE)' first to deploy the dev stack"; \
 		exit 1; \
 	fi; \
 	echo "📡 Testing endpoint: $$DEV_ENDPOINT"; \
+	if [ -n "$$IMAGE_TAG" ]; then \
+		echo "🐳 Image tag: $$IMAGE_TAG"; \
+	fi; \
+	if [ -n "$$DEPLOYED_AT" ]; then \
+		echo "📅 Deployed at: $$DEPLOYED_AT"; \
+	fi; \
 	uv run python scripts/test_webhook.py "$$DEV_ENDPOINT" --health-only
 
 # Test deployed prod stack via API Gateway endpoint
 # v0.7.0: Updated to use deployments.json with profile/stage structure
 test-deployed-prod: check-xdg
 	@echo "🧪 Testing deployed prod stack (profile: $(PROFILE))..."
 	@PROD_ENDPOINT=$$(jq -r '.active.prod.endpoint // empty' $(PROFILE_DIR)/deployments.json 2>/dev/null); \
+	IMAGE_TAG=$$(jq -r '.active.prod.imageTag // empty' $(PROFILE_DIR)/deployments.json 2>/dev/null); \
+	DEPLOYED_AT=$$(jq -r '.active.prod.timestamp // empty' $(PROFILE_DIR)/deployments.json 2>/dev/null); \
 	if [ -z "$$PROD_ENDPOINT" ]; then \
 		echo "❌ No prod endpoint found in $(PROFILE_DIR)/deployments.json"; \
 		echo "💡 Run 'npm run deploy:prod -- --profile $(PROFILE)' first to deploy the prod stack"; \
 		exit 1; \
 	fi; \
 	echo "📡 Testing endpoint: $$PROD_ENDPOINT"; \
+	if [ -n "$$IMAGE_TAG" ]; then \
+		echo "🐳 Image tag: $$IMAGE_TAG"; \
+	fi; \
+	if [ -n "$$DEPLOYED_AT" ]; then \
+		echo "📅 Deployed at: $$DEPLOYED_AT"; \
+	fi; \
 	uv run python scripts/test_webhook.py "$$PROD_ENDPOINT" --health-only
 
-# Auto-fix code formatting (black + isort)
+# Auto-fix code formatting (black + isort) and type check (pyright)
 lint:
 	uv run black src/ tests/ scripts/
 	uv run isort src/ tests/ scripts/
+	uv run pyright src/ tests/ scripts/
 
 # Deployment
 
 
@@ -2,7 +2,7 @@ manifestVersion: 1
 info:
   name: quilt-docker
   description: Packaging Benchling Notebooks as Quilt packages
-  version: 0.8.3
+  version: 0.8.4
 features:
   - name: Quilt Connector
     id: quilt_entry
 
@@ -4,9 +4,9 @@ build-backend = "hatchling.build"
 
 [project]
 name = "benchling-quilt-integration"
-version = "0.8.3"
+version = "0.8.4"
 description = "Benchling-Quilt Integration Webhook Service"
-license = {text = "MIT"}
+license = {text = "Apache-2.0"}
 authors = [
     {name = "Ernest Prabhakar", email = "ernest@quilt.bio"},
 ]
@@ -15,7 +15,7 @@ classifiers = [
     "Environment :: Web Environment",
     "Framework :: Flask",
     "Intended Audience :: Developers",
-    "License :: OSI Approved :: MIT License",
+    "License :: OSI Approved :: Apache Software License",
     "Operating System :: OS Independent",
     "Programming Language :: Python :: 3",
     "Programming Language :: Python :: 3.11",
@@ -45,6 +45,7 @@ dev = [
     "black==25.11.0",
     "flake8==7.3.0",
     "isort==7.0.0",
+    "pyright==1.1.407",
 ]
 docs = [
     "sphinx==8.2.3",
 
@@ -0,0 +1,40 @@
+{
+  "$schema": "https://raw.githubusercontent.com/microsoft/pyright/main/packages/vscode-pyright/schemas/pyrightconfig.schema.json",
+  "include": [
+    "src",
+    "tests",
+    "scripts"
+  ],
+  "exclude": [
+    "**/__pycache__",
+    "**/.pytest_cache",
+    "**/node_modules",
+    "**/.venv",
+    "**/venv"
+  ],
+  "pythonVersion": "3.11",
+  "pythonPlatform": "All",
+  "typeCheckingMode": "basic",
+  "reportMissingImports": true,
+  "reportMissingTypeStubs": false,
+  "reportGeneralTypeIssues": true,
+  "reportIncompatibleMethodOverride": true,
+  "reportIncompatibleVariableOverride": true,
+  "reportUntypedFunctionDecorator": false,
+  "reportUntypedClassDecorator": false,
+  "reportUntypedBaseClass": false,
+  "reportUnusedImport": true,
+  "reportUnusedClass": true,
+  "reportUnusedFunction": true,
+  "reportUnusedVariable": true,
+  "reportDuplicateImport": true,
+  "reportOptionalSubscript": true,
+  "reportOptionalMemberAccess": true,
+  "reportOptionalCall": true,
+  "reportOptionalIterable": true,
+  "reportOptionalContextManager": true,
+  "reportOptionalOperand": true,
+  "stubPath": "",
+  "venvPath": ".",
+  "venv": ".venv"
+}
@@ -208,14 +208,15 @@ def build(self, tag: str, platform: Optional[str] = None, version: Optional[str]
 
         os.chdir(self.project_root)
 
-        # Build docker build command
-        build_cmd = ["docker", "build", "--platform", build_platform, "--file", "Dockerfile"]
+        # Build docker buildx command (required for multi-platform builds and validation)
+        build_cmd = ["docker", "buildx", "build", "--platform", build_platform, "--file", "Dockerfile"]
 
         # Add VERSION build arg if provided
         if version:
             build_cmd.extend(["--build-arg", f"VERSION={version}"])
 
-        build_cmd.extend(["--tag", tag, "."])
+        # Use --load to load the image into Docker (instead of just building to cache)
+        build_cmd.extend(["--load", "--tag", tag, "."])
 
         result = self._run_command(build_cmd, check=False)
 
 
@@ -15,7 +15,7 @@
 
 # Add src to path for local imports
 sys.path.insert(0, os.path.join(os.path.dirname(__file__), "..", "src"))
-from config import get_config
+from config import get_config  # type: ignore
 
 # Known values to search for
 ENTRY_ID = "etr_EK1AQMQiQn"