fix(context): respect -C flag in CommandEnv worktree path (#899)

* Add OCR-based validation for TUI demos

TUI demos (Zellij, interactive UIs) can't be validated via text snapshots because
VHS only captures the outer terminal, not content inside multiplexers. Instead,
extract key frames from GIFs and use OCR (tesseract) to verify expected patterns.

Add `docs/demos/shared/validation.py` with checkpoint definitions and validation
logic. Update build script to validate TUI demos with defined checkpoints during
snapshot mode, skipping those without checkpoints. Document validation approach
in CLAUDE.md with requirements and checkpoint definitions for wt-zellij-omnibus.

* fix(demos): pre-populate Zellij permissions cache to avoid dialog

The zellij-tab-name plugin shows a permission dialog on first run,
which was appearing in recorded demos. Fix by pre-populating the
permissions.kdl cache file before Zellij starts.

Also simplifies the tape by removing the "y" keypress and clear
that were working around the permission dialog.

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

* fix(demos): re-enable OCR validation checkpoints, handle permission dialog race

The permission dialog timing is non-deterministic (depends on macOS cache state).
Handle both cases:
- If dialog appears: "y" dismisses it
- If no dialog (cache hit): "y" + Enter + clear cleans up the shell

Re-enables validation checkpoints with calibrated frame numbers:
- Frame 100: wt list output with branch table
- Frame 500: Claude UI with Opus indicator
- Frame 2000: Final wt list --full output

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

* fix(context): respect `-C` flag in CommandEnv worktree path

CommandEnv was using `std::env::current_dir()` instead of the path from
the Repository, which meant it ignored the `-C` flag. This caused hooks
to fail when running `wt -C /path/to/worktree hook ...` because
CommandContext received the wrong worktree path.

Changes:
- Add `path()` accessor to WorkingTree for getting the stored path
- Use `repo.current_worktree().path()` in for_action/for_action_branchless
- Canonicalize paths in compute_hooks_display_path() for comparison
  (handles "." vs "/absolute/path" after the fix)

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

---------

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

Author: max-sixty

src/commands/context.rs

@@ -30,7 +30,7 @@ impl CommandEnv {
     /// Used in error messages when the environment can't be loaded.
     pub fn for_action(action: &str, config: UserConfig) -> anyhow::Result<Self> {
         let repo = Repository::current()?;
-        let worktree_path = std::env::current_dir().context("Failed to get current directory")?;
+        let worktree_path = repo.current_worktree().path().to_path_buf();
         let branch = repo.require_current_branch(action)?;
 
         Ok(Self {
@@ -47,10 +47,10 @@ impl CommandEnv {
     /// such as running hooks (where `{{ branch }}` expands to "HEAD" if detached).
     pub fn for_action_branchless() -> anyhow::Result<Self> {
         let repo = Repository::current()?;
-        let worktree_path = std::env::current_dir().context("Failed to get current directory")?;
+        let current_wt = repo.current_worktree();
+        let worktree_path = current_wt.path().to_path_buf();
         // Propagate git errors (broken repo, missing git) but allow None for detached HEAD
-        let branch = repo
-            .current_worktree()
+        let branch = current_wt
             .branch()
             .context("Failed to determine current branch")?;
         let config = UserConfig::load().context("Failed to load config")?;

src/git/repository/working_tree.rs

@@ -57,6 +57,14 @@ impl<'a> WorkingTree<'a> {
         self.repo
     }
 
+    /// Get the path this WorkingTree was created with.
+    ///
+    /// This is the path passed to `worktree_at()` or `base_path()` for `current_worktree()`.
+    /// For the canonical git-determined root, use [`root()`](Self::root) instead.
+    pub fn path(&self) -> &Path {
+        &self.path
+    }
+
     /// Run a git command in this worktree and return stdout.
     pub fn run_command(&self, args: &[&str]) -> anyhow::Result<String> {
         let output = self.run_command_output(args)?;

src/output/global.rs

@@ -251,7 +251,18 @@ pub fn compute_hooks_display_path<'a>(
     hooks_run_at: &'a std::path::Path,
     user_location: &std::path::Path,
 ) -> Option<&'a std::path::Path> {
-    if hooks_run_at == user_location {
+    // Canonicalize both paths for comparison to handle relative vs absolute paths
+    // (e.g., "." vs "/absolute/path/to/cwd"). Fall back to direct comparison if
+    // canonicalization fails (e.g., path doesn't exist).
+    let same_location = match (
+        dunce::canonicalize(hooks_run_at),
+        dunce::canonicalize(user_location),
+    ) {
+        (Ok(h), Ok(u)) => h == u,
+        _ => hooks_run_at == user_location,
+    };
+
+    if same_location {
         None
     } else {
         Some(hooks_run_at)