cc-session-tools 2.7.1__tar.gz → 2.7.2__tar.gz

{cc_session_tools-2.7.1/src/cc_session_tools.egg-info → cc_session_tools-2.7.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cc-session-tools
-Version: 2.7.1
+Version: 2.7.2
 Summary: Claude Code session management CLI tools (ccd, ccr, ccs, claude-code-usage, ccst) and hook library (cccs_hooks)
 Author: raffishquartan
 License: MIT License

{cc_session_tools-2.7.1 → cc_session_tools-2.7.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "cc-session-tools"
-version = "2.7.1"
+version = "2.7.2"
 description = "Claude Code session management CLI tools (ccd, ccr, ccs, claude-code-usage, ccst) and hook library (cccs_hooks)"
 readme = "README.md"
 license = { file = "LICENSE" }

{cc_session_tools-2.7.1 → cc_session_tools-2.7.2}/src/cc_session_tools/cli/ccst.py

@@ -1071,12 +1071,14 @@ def _cmd_pdata_init(args: argparse.Namespace) -> int:
             )
         except (FileNotFoundError, ValueError) as exc:
             print(f"ccst pdata: {exc}", file=sys.stderr)
+            print(f"ERROR: {exc}")
             return 2
 
         if write_result.failure is not None:
             print("ccst pdata init: verification failed, nothing was cut over:", file=sys.stderr)
             for reason in write_result.failure.reasons:
                 print(f"  - {reason}", file=sys.stderr)
+            print(f"ERROR: verification failed ({len(write_result.failure.reasons)} reason(s))")
             return 1
 
         print(
@@ -1088,6 +1090,8 @@ def _cmd_pdata_init(args: argparse.Namespace) -> int:
             print(f"  cut over: {path}")
         print()
         print(write_result.report)  # spec §7.1 step 4's diff report, for review
+        print(f"\nVerify: ccst pdata verify --project {args.project} --full")
+        print("SUCCESS")
         return 0
 
 

{cc_session_tools-2.7.1 → cc_session_tools-2.7.2}/src/cc_session_tools/lib/pdata/backup.py

@@ -75,9 +75,12 @@ def create_backup(
     philosophy (spec goals).
 
     Written to a temp path first and atomically renamed to the final
-    `<project>-<epoch>.tar.gz` name only once the archive is fully and successfully written —
-    a failed or interrupted run never leaves a corrupt file at the name a valid backup would
-    use. Transient OSErrors (e.g. a flaky network/DrvFS-backed project_root) are retried a
+    `<project>-<YYYYMMDD-HHMMSS>.tar.gz` name only once the archive is fully and successfully
+    written — a failed or interrupted run never leaves a corrupt file at the name a valid
+    backup would use. The timestamp is second-granularity specifically so it keeps the same
+    collision-resistance across reruns within the same minute that the epoch it replaced had
+    — a coarser (e.g. minute-only) format would need a separate numeric tie-breaker to match.
+    Transient OSErrors (e.g. a flaky network/DrvFS-backed project_root) are retried a
     bounded number of times before raising BackupError. Each failed attempt is reported
     through on_progress (if given) before the retry sleep — otherwise a multi-attempt backup
     that eventually succeeds leaves no trace in --write's progress log of the retries it took,
@@ -92,7 +95,7 @@ def create_backup(
     """
     target_dir = backup_dir()
     target_dir.mkdir(parents=True, exist_ok=True)
-    final_path = target_dir / f"{project}-{int(time.time())}.tar.gz"
+    final_path = target_dir / f"{project}-{time.strftime('%Y%m%d-%H%M%S')}.tar.gz"
     tmp_path = final_path.parent / (final_path.name + ".tmp")
 
     last_exc: OSError | sqlite3.Error | None = None

{cc_session_tools-2.7.1 → cc_session_tools-2.7.2}/src/cc_session_tools/lib/pdata/write_log.py

@@ -90,8 +90,19 @@ class WriteLog:
         assert self._real_stderr is not None, "WriteLog.__exit__ called without a completed __enter__"
 
         if exc is not None:
+            # The exc_type/exc pairing is a stdlib __exit__ contract guarantee (exc_type is
+            # only ever None when exc is too), not something this method re-derives — same
+            # rationale as the asserts above.
+            assert exc_type is not None, "WriteLog.__exit__ called with exc but no exc_type"
             try:
                 traceback.print_exception(exc_type, exc, tb, file=self._file)
+                # An explicit sentinel line after the traceback, matching the ERROR/SUCCESS
+                # contract the CLI's own known exit paths write via print() — this is the one
+                # path that never reaches back to those print() calls, since the exception
+                # escapes the `with` block entirely. Without it, an absent sentinel would mean
+                # two different things (an escaped exception vs. the process being killed
+                # before __exit__ ran) instead of just the latter.
+                self._file.write(f"ERROR: {exc_type.__name__}: {exc}\n")
                 self._file.flush()
             except OSError:
                 pass  # same best-effort contract as _Tee — never let the log itself crash

{cc_session_tools-2.7.1 → cc_session_tools-2.7.2/src/cc_session_tools.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cc-session-tools
-Version: 2.7.1
+Version: 2.7.2
 Summary: Claude Code session management CLI tools (ccd, ccr, ccs, claude-code-usage, ccst) and hook library (cccs_hooks)
 Author: raffishquartan
 License: MIT License

{cc_session_tools-2.7.1 → cc_session_tools-2.7.2}/tests/test_ccst_pdata_init_cli.py

@@ -90,11 +90,17 @@ def test_pdata_init_write_end_to_end_imports_and_cuts_over(base_env, tmp_path):
 
 
 def test_pdata_init_write_without_prior_dry_run_errors(base_env, tmp_path):
-    (tmp_path / "projects" / "demo").mkdir(parents=True)
+    project_dir = tmp_path / "projects" / "demo"
+    project_dir.mkdir(parents=True)
     r = _run(base_env, "pdata", "init", "--project", "demo", "--write")
     assert r.returncode == 2
     assert "proposal" in r.stderr
 
+    # This is the FileNotFoundError/ValueError branch inside WriteLog's `with` block (project
+    # root resolution already succeeded), so it must still end in the ERROR sentinel.
+    log_content = (project_dir / "ccst-pdata-init-write.log").read_text()
+    assert log_content.rstrip().splitlines()[-1].startswith("ERROR: ")
+
 
 def test_pdata_init_write_aborts_on_bad_file_path_without_cutover(base_env, tmp_path):
     project_dir = tmp_path / "projects" / "demo"
@@ -158,6 +164,43 @@ def test_pdata_init_write_log_captures_verification_failure(base_env, tmp_path):
     assert "verification failed" in log_content.lower()
 
 
+def test_pdata_init_write_success_ends_with_success_sentinel_and_verify_command(base_env, tmp_path):
+    project_dir = tmp_path / "projects" / "demo"
+    project_dir.mkdir(parents=True)
+    (project_dir / "ideas.csv").write_text("idea\nfirst\nsecond\n")
+    base_env["CCST_PDATA_BACKUP_DIR"] = str(tmp_path / "backups")
+
+    _run(base_env, "pdata", "init", "--project", "demo")
+    r_write = _run(base_env, "pdata", "init", "--project", "demo", "--write")
+
+    assert r_write.returncode == 0, r_write.stderr
+    assert "ccst pdata verify --project demo --full" in r_write.stdout
+    assert r_write.stdout.rstrip().splitlines()[-1] == "SUCCESS"
+
+    log_content = (project_dir / "ccst-pdata-init-write.log").read_text()
+    assert log_content.rstrip().splitlines()[-1] == "SUCCESS"
+
+
+def test_pdata_init_write_verification_failure_ends_with_error_sentinel(base_env, tmp_path):
+    project_dir = tmp_path / "projects" / "demo"
+    project_dir.mkdir(parents=True)
+    (project_dir / "docs.csv").write_text("doc_path,note\n/etc/passwd,bad\n")
+    base_env["CCST_PDATA_BACKUP_DIR"] = str(tmp_path / "backups")
+
+    _run(base_env, "pdata", "init", "--project", "demo")
+    proposal_path = project_dir / ".ccst-pdata-proposal.json"
+    data = json.loads(proposal_path.read_text())
+    data["entries"][0]["file_path_column"] = "doc_path"
+    proposal_path.write_text(json.dumps(data))
+
+    r_write = _run(base_env, "pdata", "init", "--project", "demo", "--write")
+
+    assert r_write.returncode == 1
+    assert r_write.stdout.rstrip().splitlines()[-1].startswith("ERROR: ")
+    log_content = (project_dir / "ccst-pdata-init-write.log").read_text()
+    assert log_content.rstrip().splitlines()[-1].startswith("ERROR: ")
+
+
 def test_pdata_init_rehearse_does_not_touch_real_project(base_env, tmp_path):
     project_dir = tmp_path / "projects" / "demo"
     project_dir.mkdir(parents=True)