basic-memory 0.23.1.dev27__py3-none-any.whl → 0.23.2__py3-none-any.whl

basic_memory/__init__.py

@@ -1,7 +1,7 @@
 """basic-memory - Local-first knowledge management combining Zettelkasten with knowledge graphs"""
 
 # Package version - updated by release automation
-__version__ = "0.23.0"
+__version__ = "0.23.2"
 
 # API version for FastAPI - independent of package version
 __api_version__ = "v2"

basic_memory/indexing/accepted_note_mutation_runner.py

@@ -60,6 +60,7 @@ from basic_memory.runtime.storage import (
 )
 from basic_memory.schemas.base import Entity as EntitySchema
 from basic_memory.schemas.request import EditEntityRequest
+from basic_memory.utils import resolve_directory_casing
 
 type AcceptedNoteMutationChange = RuntimeAcceptedNoteChange[RuntimeNoteContentResponsePayload]
 type AcceptedNoteMutationUserProfileId = UUID
@@ -233,6 +234,11 @@ class AcceptedNoteMutationEntityRepository(Protocol):
         load_relations: bool = False,
     ) -> Entity | None: ...
 
+    async def get_distinct_directories(
+        self,
+        session: AsyncSession,
+    ) -> list[str]: ...
+
 
 class AcceptedNoteMutationNoteContentRepository(Protocol):
     """note_content lookup capability for accepted-note mutations."""
@@ -379,6 +385,58 @@ async def resolve_accepted_note_source_checksum(
     )
 
 
+async def resolve_accepted_note_directory(
+    session: AsyncSession,
+    *,
+    project_id: ProjectId,
+    directory: str,
+    dependencies: AcceptedNoteMutationDependencies,
+) -> str:
+    """Resolve a requested note directory against existing folder casing (#1326).
+
+    Trigger: the requested directory is not an existing folder but matches
+        exactly one existing folder case-insensitively.
+    Why: LLM callers guess plausible casing ("schemas" beside an existing
+        "Schemas/"); on case-sensitive storage (cloud object storage) the guess
+        silently creates a case-duplicate sibling folder. Folders are derived
+        from indexed entity file paths in the DB, never by probing storage, so
+        local and cloud runtimes resolve identically.
+    Outcome: a unique case-insensitive match adopts the existing folder's
+        casing; exact matches, unknown folders, and ambiguous case-variant
+        siblings keep the requested casing unchanged.
+    """
+    if not directory:
+        return directory
+    entity_repository = dependencies.lookup_repositories.entity_repository(project_id)
+    existing_directories = await entity_repository.get_distinct_directories(session)
+    return resolve_directory_casing(directory, existing_directories)
+
+
+async def resolve_accepted_note_schema_directory(
+    session: AsyncSession,
+    *,
+    project_id: ProjectId,
+    data: EntitySchema,
+    dependencies: AcceptedNoteMutationDependencies,
+) -> EntitySchema:
+    """Return the write schema with its directory resolved to existing casing.
+
+    The schema's ``file_path`` is computed from ``directory``, so adjusting the
+    directory redirects the conflict lookup, preparation, and permalink
+    resolution downstream. The caller's schema is never mutated: a resolved
+    directory yields a copy so route-owned request data stays as received.
+    """
+    resolved_directory = await resolve_accepted_note_directory(
+        session,
+        project_id=project_id,
+        directory=data.directory,
+        dependencies=dependencies,
+    )
+    if resolved_directory == data.directory:
+        return data
+    return data.model_copy(update={"directory": resolved_directory})
+
+
 async def run_accepted_note_create(
     session: AsyncSession,
     *,
@@ -501,10 +559,16 @@ async def _run_accepted_note_create(
         dependencies=dependencies,
     )
 
+    data = await resolve_accepted_note_schema_directory(
+        session,
+        project_id=project.id,
+        data=request.data,
+        dependencies=dependencies,
+    )
     entity_repository = dependencies.lookup_repositories.entity_repository(project.id)
     conflicting_entity = await entity_repository.get_by_file_path(
         session,
-        request.data.file_path,
+        data.file_path,
         load_relations=False,
     )
     reject_accepted_note_file_path_conflict(
@@ -515,7 +579,7 @@ async def _run_accepted_note_create(
     preparer = dependencies.preparer_factory.create_note_preparer(project)
     prepared_write = await prepare_create_or_reject(
         preparer,
-        request.data,
+        data,
         check_storage_exists=dependencies.verify_storage_absent_on_create,
         session=session,
     )
@@ -579,10 +643,26 @@ async def _run_accepted_note_update(
     existing_file_path = entity.file_path if entity is not None else None
     vacated_source: tuple[RuntimeFilePath, RuntimeFileChecksum | None] | None = None
 
+    # Trigger: the addressed entity already lives in the exact requested directory.
+    # Why: casing resolution scans every distinct entity file_path in the project,
+    #     and content-only PUTs (e.g. repeated collaboration-relay saves of the
+    #     same note) are the hot path where the directory cannot change; an exact
+    #     match would resolve to itself anyway because exact match always wins.
+    # Outcome: only case-variant or relocating PUTs pay for the directory scan.
+    current_directory = existing_file_path.rpartition("/")[0] if existing_file_path else None
+    if current_directory == request.data.directory:
+        data = request.data
+    else:
+        data = await resolve_accepted_note_schema_directory(
+            session,
+            project_id=project.id,
+            data=request.data,
+            dependencies=dependencies,
+        )
     await reject_conflicting_accepted_note_file_path(
         session,
         project_id=project.id,
-        file_path=request.data.file_path,
+        file_path=data.file_path,
         allowed_entity_external_id=request.entity_external_id,
         dependencies=dependencies,
     )
@@ -600,7 +680,7 @@ async def _run_accepted_note_update(
             reject_stale_base_checksum(current_db_checksum=None)
         prepared_write = await prepare_create_or_reject(
             preparer,
-            request.data,
+            data,
             check_storage_exists=dependencies.verify_storage_absent_on_create,
             session=session,
         )
@@ -630,7 +710,7 @@ async def _run_accepted_note_update(
             try:
                 await preparer.verify_move_destination_absent(
                     source_file_path=entity.file_path,
-                    destination_file_path=request.data.file_path,
+                    destination_file_path=data.file_path,
                 )
             except EntityAlreadyExistsError as error:
                 reject_accepted_note_mutation(AcceptedNoteMutationRejectKind.conflict, str(error))
@@ -644,7 +724,7 @@ async def _run_accepted_note_update(
         # A PUT replacement may also rename the note. Capture the exact source bytes before
         # persistence mutates the entity and note_content to the destination version so delayed
         # cleanup cannot let a later project index recreate the old path as a ghost.
-        if request.data.file_path != entity.file_path:
+        if data.file_path != entity.file_path:
             vacated_source = (
                 entity.file_path,
                 await resolve_accepted_note_source_checksum(
@@ -693,7 +773,7 @@ async def _run_accepted_note_update(
                 preparer,
                 session,
                 entity=entity,
-                data=request.data,
+                data=data,
                 current_note_content=current_note_content,
                 user_profile_value=user_profile_value,
             )
@@ -825,6 +905,18 @@ async def _run_accepted_note_move(
         dependencies=dependencies,
     )
     existing_file_path = entity.file_path
+    # The destination filename keeps its requested casing; only the parent
+    # directory resolves against existing folders (issue #1326). The path is
+    # posix-normalized above, so rpartition splits directory from filename.
+    destination_directory, _, destination_filename = accepted_file_path.rpartition("/")
+    resolved_directory = await resolve_accepted_note_directory(
+        session,
+        project_id=project.id,
+        directory=destination_directory,
+        dependencies=dependencies,
+    )
+    if resolved_directory != destination_directory:
+        accepted_file_path = f"{resolved_directory}/{destination_filename}"
     # Same-path moves fail fast everywhere by decision (2026-07-14): cloud's
     # pre-unification route returned a 200 no-op, local rejected — the unified
     # runner keeps the rejection so a mistaken identity move surfaces instead

basic_memory/mcp/tools/move_note.py

@@ -947,9 +947,20 @@ move_note("{identifier}", destination_folder="notes")
             #      the up-front detection: any divergence the caller did not ask for
             #      surfaces as a failure rather than a fake success.
             # Outcome: report failure with the actual landing path instead of "✅".
+            # A case-only difference in the PARENT directories is the server resolving
+            # the destination folder to an existing folder's casing (#1326), not a
+            # boundary degradation — the success message below reports the actual
+            # landing path either way. The server always preserves the requested
+            # filename verbatim, so a basename divergence (even case-only) still
+            # reports the honest failure.
             normalized_requested = PureWindowsPath(destination_path).as_posix().strip("/")
             normalized_actual = PureWindowsPath(result.file_path).as_posix().strip("/")
-            if normalized_actual != normalized_requested:
+            requested_parent, _, requested_name = normalized_requested.rpartition("/")
+            actual_parent, _, actual_name = normalized_actual.rpartition("/")
+            diverged = normalized_actual != normalized_requested and not (
+                actual_name == requested_name and actual_parent.lower() == requested_parent.lower()
+            )
+            if diverged:
                 logger.warning(
                     f"Move outcome diverged from intent: requested={destination_path} "
                     f"actual={result.file_path}"

basic_memory/utils.py

@@ -776,6 +776,52 @@ def detect_potential_file_conflicts(file_path: str, existing_paths: List[str]) -
     return conflicts
 
 
+def resolve_directory_casing(directory: str, existing_directories: List[str]) -> str:
+    """Resolve a requested directory to the casing of existing folders (#1326).
+
+    LLM callers guess plausible-but-wrong casing ("schemas" beside an existing
+    "Schemas/"), and case-sensitive storage backends then grow case-duplicate
+    sibling folders. Each path segment resolves against the folders that exist
+    under the already-resolved parent:
+
+    - Exact match wins: the requested segment casing is kept unchanged.
+    - Unique case-insensitive match: the existing folder's casing is adopted.
+    - No match: the segment is kept as given (a new folder).
+    - Multiple case-variant folders exist: ambiguous, the segment is kept as
+      given (today's exact behavior).
+
+    Args:
+        directory: Requested directory path relative to the project root
+            ("" means the root itself).
+        existing_directories: Known project folders derived from indexed entity
+            file paths (see EntityRepository.get_distinct_directories).
+
+    Returns:
+        The directory path with each segment resolved to existing casing.
+    """
+    if not directory:
+        return directory
+
+    existing = set(existing_directories)
+    resolved_prefix = ""
+    for segment in directory.split("/"):
+        candidate = f"{resolved_prefix}/{segment}" if resolved_prefix else segment
+        if candidate in existing:
+            resolved_prefix = candidate
+            continue
+        # Candidates are constrained to children of the already-resolved parent:
+        # once a parent segment stays ambiguous (kept as requested), a deeper
+        # existing folder under a *different* parent casing must not be spliced in.
+        matches = [
+            existing_directory
+            for existing_directory in existing
+            if existing_directory.lower() == candidate.lower()
+            and existing_directory.rpartition("/")[0] == resolved_prefix
+        ]
+        resolved_prefix = matches[0] if len(matches) == 1 else candidate
+    return resolved_prefix
+
+
 def valid_project_path_value(path: str):
     """Ensure project path is valid."""
     # Allow empty strings as they resolve to the project root

{basic_memory-0.23.1.dev27.dist-info → basic_memory-0.23.2.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: basic-memory
-Version: 0.23.1.dev27
+Version: 0.23.2
 Summary: Local-first knowledge management combining Zettelkasten with knowledge graphs
 Project-URL: Homepage, https://github.com/basicmachines-co/basic-memory
 Project-URL: Repository, https://github.com/basicmachines-co/basic-memory

{basic_memory-0.23.1.dev27.dist-info → basic_memory-0.23.2.dist-info}/RECORD

@@ -1,4 +1,4 @@
-basic_memory/__init__.py,sha256=yREnsMqgfnZJBuh1hZFPgZv-ERBpfL361wFDx9sdijA,256
+basic_memory/__init__.py,sha256=dtmjVy3Q4_DYddSDwS9XOq3okErGMfYtVZ-_1s5p2zQ,256
 basic_memory/config.py,sha256=gPw1piQZJANYCF-TkpEpc9-vD7-lHnSez8HQidGh5qk,12080
 basic_memory/config_logging.py,sha256=keqWv5xPOP6fjh1LDQmKMDyduHTYAhr6Y1EKiiYVDnc,1857
 basic_memory/config_migrations.py,sha256=OO5hwzs3x5EEix-jNMeETrJQwMx0l2iHUywXRLMjzFA,3582
@@ -10,7 +10,7 @@ basic_memory/migration_loop.py,sha256=KGhuCLgrgwW21GAGT9cOczbtRlM4mGi7aUBjG1shQ8
 basic_memory/project_resolver.py,sha256=vHWHw1X9_1IqtX9Dy5yNXqNRas3wcrpQJVKxJuj4qWg,3801
 basic_memory/redaction.py,sha256=tYolNeRQdOfcELD9sGtzQbB5dS3odP9PAnzipqmMl6M,4804
 basic_memory/telemetry.py,sha256=pAkPgbAOmivZDkCAW0S2CKSNw4AqBfMckdsbb2EipcQ,1721
-basic_memory/utils.py,sha256=TIv_NtIXJKRyCsGUX_0nWWz65hdiqK_LZ17zq-r0q08,33363
+basic_memory/utils.py,sha256=bXlhv3qHOO_1Lb_2gUspjOZ3H1-qgreWl_n5eR6_Jxg,35426
 basic_memory/workspace_context.py,sha256=v-u4jO8VFB1WeFZ1UpzZnbf9ph71vvz1OMibpAtrtQI,3648
 basic_memory/alembic/alembic.ini,sha256=IEZsnF8CbbZnkwBr67LzKKNobHuzTaQNUvM8Psop5xc,3733
 basic_memory/alembic/env.py,sha256=ZfUWoJeIJKTCsOSvz6mty0c570k2k4MF9cvi7T4sj1g,9581
@@ -160,7 +160,7 @@ basic_memory/index/watch_coordinator.py,sha256=Nif22CFD8romuHTdPTTW1LHyKvy2UyUbv
 basic_memory/index/watch_service.py,sha256=R4uOFjLVTQjSYo1fE6kOUcnnq3o7JPIlYQFZf0PjXCI,13885
 basic_memory/indexing/__init__.py,sha256=YdS2QwyyQ7fPxvUr2wkubztu2aolAyuQy9Y0FbH0BH8,236
 basic_memory/indexing/accepted_note_enqueue_runner.py,sha256=v0BQTdKFOnhwrvls4QBqF2fIMejRkeFWyQpimOdoUt4,7540
-basic_memory/indexing/accepted_note_mutation_runner.py,sha256=NDa_SrxFT_f_Fj1JSQY-c0Uyk51TQiwJqInt1E9UPck,43587
+basic_memory/indexing/accepted_note_mutation_runner.py,sha256=BTaMDcE2GpF4S5uhF2tHzxFbBOy1fOgUVjmB2_Wyj0M,47564
 basic_memory/indexing/accepted_note_search.py,sha256=fNcuxOKJ_zVM6FVlZQ8-OGjG3E3NXGJwkTuvO8aF0PU,4748
 basic_memory/indexing/accepted_note_write_runner.py,sha256=ZvQlX47C8oD-VTCBWSEDCzTE3x938IPetZQp48_H_PM,25759
 basic_memory/indexing/batch_indexer.py,sha256=Yy9CcZcHzUsLG24aAsaUVrCjkDFemAQeagDmr8IFV_s,37976
@@ -245,7 +245,7 @@ basic_memory/mcp/tools/chatgpt_tools.py,sha256=BpBLzT4zGe7TMLoFWbbnTq9lwcVS_1_F1
 basic_memory/mcp/tools/delete_note.py,sha256=RPFXo1QxxMV6PFw4Uf9eXdOBUpn_m8rJkVr4pFSVflI,20822
 basic_memory/mcp/tools/edit_note.py,sha256=owbnPnncW__O0cBAR0sZcLo37b4tTFtFtNYatTVhiGA,44695
 basic_memory/mcp/tools/list_directory.py,sha256=YQT4AmEfbmyHOWGYlU0s-htNBvZBzonA7gZQ-btiNkg,10367
-basic_memory/mcp/tools/move_note.py,sha256=bP_nYFCWQsqmiCe0uBNl1rJoX2b3rZiNXXxfjQ0MIJo,44360
+basic_memory/mcp/tools/move_note.py,sha256=DzsB7sQU0Q-KlVGELN3kb9ivXfbqEiVNfFTNzxVFoqc,45139
 basic_memory/mcp/tools/project_management.py,sha256=B2MCcX7PCjAobfJkDCI5KiJkT4AVE43yNXGpZoICSpU,36341
 basic_memory/mcp/tools/read_content.py,sha256=eaGy4bV1fF7KAoyP9QgautDsaJUR2u8GOqx0CG0p5Ug,12970
 basic_memory/mcp/tools/read_note.py,sha256=BS40PBOSHwf7LYdfZK1OiNhsJvOSccen3V81UR6AG_0,25636
@@ -391,8 +391,8 @@ basic_memory/services/search_service.py,sha256=8A1vXo5IIpKXL2gxEsCEP_tcKyRcnz4R5
 basic_memory/services/service.py,sha256=V-d_8gOV07zGIQDpL-Ksqs3ZN9l3qf3HZOK1f_YNTag,336
 basic_memory/templates/prompts/continue_conversation.hbs,sha256=trrDHSXA5S0JCbInMoUJL04xvCGRB_ku1RHNQHtl6ZI,3076
 basic_memory/templates/prompts/search.hbs,sha256=H1cCIsHKp4VC1GrH2KeUB8pGe5vXFPqb2VPotypmeCA,3098
-basic_memory-0.23.1.dev27.dist-info/METADATA,sha256=j0HNhCqLZhDGmlKdwxHIf7eWLNDXeKyhOgF8UEzKrPI,26848
-basic_memory-0.23.1.dev27.dist-info/WHEEL,sha256=zOwg4jB6zX2kU910N-cMawjivD6tO8NEWvE12je1bVk,87
-basic_memory-0.23.1.dev27.dist-info/entry_points.txt,sha256=wvE2mRF6-Pg4weIYcfQ-86NOLZD4WJg7F7TIsRVFLb8,90
-basic_memory-0.23.1.dev27.dist-info/licenses/LICENSE,sha256=hIahDEOTzuHCU5J2nd07LWwkLW7Hko4UFO__ffsvB-8,34523
-basic_memory-0.23.1.dev27.dist-info/RECORD,,
+basic_memory-0.23.2.dist-info/METADATA,sha256=wy0hbehVXk8xFwvXvdqv-oqzkmO8KyEbWzgqbZvP-Ps,26842
+basic_memory-0.23.2.dist-info/WHEEL,sha256=zOwg4jB6zX2kU910N-cMawjivD6tO8NEWvE12je1bVk,87
+basic_memory-0.23.2.dist-info/entry_points.txt,sha256=wvE2mRF6-Pg4weIYcfQ-86NOLZD4WJg7F7TIsRVFLb8,90
+basic_memory-0.23.2.dist-info/licenses/LICENSE,sha256=hIahDEOTzuHCU5J2nd07LWwkLW7Hko4UFO__ffsvB-8,34523
+basic_memory-0.23.2.dist-info/RECORD,,