fix: file catalog URLs missing /latest/ version component (#314)

* fix(packages): include /latest/ version in file catalog URLs

Fixed file URL generation to include the version component in the path.
File URLs now follow the correct format:
/tree/latest/{file_path} instead of /tree/{file_path}

Example:
- Before: .../tree/README.md
- After: .../tree/latest/README.md

Updated tests to verify the correct URL format and ensure the fix
prevents regression.

Fixes #313

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

* chore: bump version to 0.9.8

* docs: add CHANGELOG entry for v0.9.8

Document fix for file catalog URLs missing /latest/ version component.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

---------

Co-authored-by: Claude <noreply@anthropic.com>

Author: drernie

CHANGELOG.md

@@ -3,6 +3,16 @@
 
 All notable changes to this project will be documented in this file, one line per user-visible change.
 
+## [0.9.8] - 2025-12-22
+
+### Fixed
+
+- **File catalog URLs now include version component** - Fixed file URL generation to include `/latest/` in path (#313)
+  - File URLs now follow correct format: `/tree/latest/{file_path}` instead of `/tree/{file_path}`
+  - Example: `https://catalog.com/b/bucket/packages/benchling/EXP25000088/tree/latest/README.md`
+  - Updated tests to verify correct URL format and prevent regression
+  - Affects `Package.make_catalog_url()` and `PackageFile.catalog_url` property
+
 ## [0.9.7] - 2025-12-18
 
 ### Fixed

docker/app-manifest.yaml

@@ -2,7 +2,7 @@ manifestVersion: 1
 info:
   name: quilt-docker
   description: Packaging Benchling Notebooks as Quilt packages
-  version: 0.9.7
+  version: 0.9.8
 features:
   - name: Quilt Connector
     id: quilt_entry

docker/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "benchling-quilt-integration"
-version = "0.9.7"
+version = "0.9.8"
 description = "Benchling-Quilt Integration Webhook Service"
 license = {text = "Apache-2.0"}
 authors = [

docker/src/packages.py

@@ -70,11 +70,11 @@ def make_catalog_url(self, logical_key: str) -> str:
             Direct link to file in Quilt catalog
 
         Example:
-            'https://nightly.quilttest.com/b/my-bucket/packages/benchling/etr_123/tree/data%2Ffile.csv'
+            'https://nightly.quilttest.com/b/my-bucket/packages/benchling/etr_123/tree/latest/data%2Ffile.csv'
         """
         encoded_key = quote(logical_key, safe="")
         base_url = f"https://{self.catalog_base_url}/b/{self.bucket}/packages/{self.package_name}"
-        return f"{base_url}/tree/{encoded_key}"
+        return f"{base_url}/tree/latest/{encoded_key}"
 
     def make_sync_url(self, path: Optional[str] = None, version: Optional[str] = None) -> str:
         """Generate QuiltSync download URL for the package or a file within it.

docker/tests/test_package_files.py

@@ -67,7 +67,7 @@ def test_package_file_size_display(self):
         assert file.size_display == "1.0 GB"
 
     def test_package_file_catalog_url(self):
-        """Test catalog URL generation."""
+        """Test catalog URL generation includes /latest/ version."""
         file = PackageFile(
             logical_key="data/exp.csv",
             size=1024,
@@ -80,7 +80,7 @@ def test_package_file_catalog_url(self):
         assert "https://nightly.quilttest.com" in url
         assert "/b/test-bucket" in url
         assert "/packages/benchling/etr_123" in url
-        assert "/tree/" in url
+        assert "/tree/latest/" in url
 
     def test_package_file_sync_url(self):
         """Test QuiltSync URL generation."""

docker/tests/test_packages.py

@@ -92,25 +92,25 @@ def test_upload_url_is_property(self, package):
         assert "?action=revisePackage" in result
 
     def test_make_catalog_url_with_file_path(self, package):
-        """Test make_catalog_url generates correct URL for files."""
+        """Test make_catalog_url generates correct URL for files with /latest/ version."""
         result = package.make_catalog_url("README.md")
-        expected = "https://nightly.quilttest.com/b/test-bucket/packages/benchling/etr_123/tree/README.md"
+        expected = "https://nightly.quilttest.com/b/test-bucket/packages/benchling/etr_123/tree/latest/README.md"
         assert result == expected
 
     def test_make_catalog_url_with_nested_path(self, package):
-        """Test make_catalog_url with nested file paths."""
+        """Test make_catalog_url with nested file paths includes /latest/ version."""
         result = package.make_catalog_url("data/results.csv")
-        # Forward slash should be encoded
-        assert "/tree/data%2Fresults.csv" in result
+        # Forward slash should be encoded and version should be included
+        assert "/tree/latest/data%2Fresults.csv" in result
 
     def test_make_catalog_url_special_characters(self, package):
-        """Test make_catalog_url encodes special characters in file paths."""
+        """Test make_catalog_url encodes special characters in file paths and includes /latest/ version."""
         result = package.make_catalog_url("data/file name with spaces.txt")
-        # Spaces should be encoded
+        # Spaces should be encoded and version should be included
         assert "%20" in result
         expected_url = (
             "https://nightly.quilttest.com/b/test-bucket/packages/benchling/etr_123/"
-            "tree/data%2Ffile%20name%20with%20spaces.txt"
+            "tree/latest/data%2Ffile%20name%20with%20spaces.txt"
         )
         assert result == expected_url
 

docker/uv.lock

@@ -1,6 +1,6 @@
 {
   "name": "@quiltdata/benchling-webhook",
-  "version": "0.9.7",
+  "version": "0.9.8",
   "description": "AWS CDK deployment for Benchling webhook processing using Fargate - Deploy directly with npx",
   "main": "dist/lib/index.js",
   "types": "dist/lib/index.d.ts",