refactor: consolidate constants, add documentation, and create tests

Config consolidation:
- Rename constants for consistency (DirContext, DirArchive, etc.)
- Add update type constants (UpdateTypeTask, UpdateTypeDecision, etc.)
- Add WatchAutoSaveInterval constant
- Use filepath.Join for all path construction

Documentation:
- Add godoc comments with Parameters/Returns/Fields sections
- Add copyright headers to all watch package files
- Add package doc comment to watch package

Tests:
- Add watch_test.go with tests for applyUpdate, complete, stream, session
- Add colocated tests for add, agent, compact, complete, drift, hook,
  initialize, load, loop, session, status, sync, task, validation packages
- Slim cli_test.go to only keep TestBinaryIntegration

Code improvements:
- Use config constants instead of literal strings throughout codebase
- Fix block-non-path-ctx.sh regex to not overfit on paths containing "ctx"
- Rename init package to initialize (canonical Go naming)

Signed-off-by: Jose Alekhinne <alekhinejose@gmail.com>

Author: josealekhine

.claude/hooks/auto-save-session.sh

@@ -35,7 +35,7 @@ PROJECT_DIR=$(echo "$HOOK_INPUT" | jq -r '.cwd // "."')
 
 # Only proceed if we have a transcript path
 if [ -z "$TRANSCRIPT_PATH" ] || [ ! -f "$TRANSCRIPT_PATH" ]; then
-    exit 0
+  exit 0
 fi
 
 # Create sessions directory if it doesn't exist

.claude/hooks/block-non-path-ctx.sh

@@ -25,37 +25,38 @@ COMMAND=$(echo "$HOOK_INPUT" | jq -r '.tool_input.command // empty')
 
 # If no command, allow (not a Bash call we care about)
 if [ -z "$COMMAND" ]; then
-    exit 0
+  exit 0
 fi
 
 # Check for forbidden patterns
 BLOCKED_REASON=""
 
 # Pattern 1: ./ctx or ./dist/ctx or ./dist/ctx-*
 if echo "$COMMAND" | grep -qE '(\./ctx|\./dist/ctx)'; then
-    BLOCKED_REASON="Use 'ctx' from PATH, not './ctx' or './dist/ctx'. Install with: sudo make install"
+  BLOCKED_REASON="Use 'ctx' from PATH, not './ctx' or './dist/ctx'. Install with: sudo make install"
 fi
 
 # Pattern 2: go run ./cmd/ctx
 if echo "$COMMAND" | grep -qE 'go run \./cmd/ctx'; then
-    BLOCKED_REASON="Use 'ctx' from PATH, not 'go run ./cmd/ctx'. Install with: sudo make install"
+  BLOCKED_REASON="Use 'ctx' from PATH, not 'go run ./cmd/ctx'. Install with: sudo make install"
 fi
 
 # Pattern 3: Absolute paths to ctx binary (but not just 'ctx' or paths in /usr/local/bin, /usr/bin)
-# Match things like /home/user/project/ctx or /tmp/ctx-test but allow /usr/local/bin/ctx
-if echo "$COMMAND" | grep -qE '(/home/|/tmp/|/var/)[^ ]*ctx[^ ]* '; then
-    # Exception: allow /tmp/ctx-test for integration tests
-    if ! echo "$COMMAND" | grep -qE '/tmp/ctx-test'; then
-        BLOCKED_REASON="Use 'ctx' from PATH, not absolute paths. Install with: sudo make install"
-    fi
+# Match things like /home/user/project/ctx or /tmp/ctx but allow /usr/local/bin/ctx
+# Only match when ctx is the binary (end of path), not a directory in the middle of a path
+if echo "$COMMAND" | grep -qE '(/home/|/tmp/|/var/)[^ ]*/ctx( |$)'; then
+  # Exception: allow /tmp/ctx-test for integration tests
+  if ! echo "$COMMAND" | grep -qE '/tmp/ctx-test'; then
+    BLOCKED_REASON="Use 'ctx' from PATH, not absolute paths. Install with: sudo make install"
+  fi
 fi
 
 # If blocked, output JSON that tells Claude Code to reject
 if [ -n "$BLOCKED_REASON" ]; then
-    cat << EOF
+  cat << EOF
 {"decision": "block", "reason": "$BLOCKED_REASON\n\nSee CONSTITUTION.md: ctx Invocation Invariants"}
 EOF
-    exit 0
+  exit 0
 fi
 
 # Allow the command

.context/TASKS.md

@@ -1,49 +1,128 @@
 # Tasks — Context CLI
 
-## In Progress
+# Tasks
 
-## Phase 1
+### Phase 1: Parser
 
-- [ ] Verify all Markdown files by "actually reading them"; take notes for
-  follow-up actions.
-- [ ] All go code should have godoc and testing.
-- [ ] GitHub CI linter is giving errors that need fixing.
-- [ ] Manual code review. take notes.
-- [ ] Add tests per file.
-- [ ] validate everything in the docs with a skeptical eye.
-- [ ] consider the case where `ctx` is not called from within AI prompt:
-  - does the command still make sense?
-  - does it create the expected output?
-- [ ] Cut the first release.
+- [ ] T1.1.1: Define data structures in `internal/recall/parser/types.go`
+  - `SessionMessage`, `Session`, `ContentBlock` interface
+  - `TextBlock`, `ThinkingBlock`, `ToolUseBlock`, `ToolResultBlock`
+  - `TokenUsage` struct
+  - Must match `specs/recall/v0.1.0/01-session-schema.md`
+
+- [ ] T1.1.2: Implement line parser in `internal/recall/parser/parser.go`
+  - `ParseLine(line []byte) (*SessionMessage, error)`
+  - Handle malformed JSON (return error, don't panic)
+  - Handle missing optional fields with defaults
+
+- [ ] T1.1.3: Implement session grouper
+  - `ParseFile(path string) (map[string]*Session, error)`
+  - Stream lines, don't load entire file into memory
+  - Sort messages by timestamp within each session
+  - Calculate: TurnCount, TotalTokensIn, TotalTokensOut, Duration
+
+- [ ] T1.1.4: Implement directory scanner
+  - `ScanDirectory(path string) ([]*Session, error)`
+  - Recurse into subdirectories
+  - Skip non-JSONL files
+  - Aggregate sessions across files
+
+### Phase 2: Renderer
+
+- [ ] T1.2.1: Set up template system in `internal/recall/renderer/`
+  - Use `//go:embed` for templates
+  - Create `templates/layout.html` with dark mode CSS
+  - Create `templates/index.html`
+  - Create `templates/session.html`
+
+- [ ] T1.2.2: Implement markdown renderer
+  - Add goldmark + chroma dependencies
+  - `RenderMarkdown(text string) template.HTML`
+  - Enable GFM extensions
+  - Syntax highlighting with monokai theme
+
+- [ ] T1.2.3: Implement session renderer
+  - `RenderSession(session *Session) (*RenderedSession, error)`
+  - Wrap thinking blocks in `<details>` (collapsed by default)
+  - Format tool calls with syntax-highlighted input/output
 
+- [ ] T1.2.4: Implement index renderer
+  - `RenderIndex(sessions []*Session, filters Filters) (*RenderedIndex, error)`
+  - Sort by date (newest first)
+  - Include preview (first 100 chars of first user message)
+  - Show aggregate stats
 
-## Next Up
+### Phase 3: Server
 
-- [ ] Enforce test coverage targets in CI/Makefile #priority:medium #area:quality
-  - internal/cli: 60% (currently 62.8%)
-  - internal/context: 80% (currently 86.8%)
-  - internal/drift: 80% (currently 88.0%)
-  - internal/claude: 80% (currently 87.5%)
-  - internal/templates: 80% (currently 88.9%)
+- [ ] T1.3.1: Set up HTTP server in `internal/recall/server/`
+  - Standard library `net/http`
+  - Graceful shutdown on SIGINT/SIGTERM
+  - Embed static assets with `//go:embed`
 
-## Completed (Recent)
+- [ ] T1.3.2: Implement index route
+  - `GET /` — render index page
+  - `GET /?project=X` — filter by project
+  - `GET /?after=DATE&before=DATE` — filter by date
+  - `GET /?q=QUERY` — search sessions
 
-## Blocked
+- [ ] T1.3.3: Implement session detail route
+  - `GET /session/:id` — render session detail
+  - 404 if not found
+  - Include back link to index
 
-## Reference
+- [ ] T1.3.4: Implement API routes
+  - `GET /api/sessions` — JSON session list
+  - `GET /api/session/:id` — JSON session detail
 
-**Specs** (in `specs/` directory):
-- `core-architecture.md` — Overall design philosophy
-- `go-cli-implementation.md` — Go project structure and patterns
-- `cli.md` — All CLI commands and their behavior
-- `context-file-formats.md` — File format specifications
-- `context-loader.md` — Loading and parsing logic
-- `context-updater.md` — Update command handling
+### Phase 4: Search
 
-**Task Status Labels**:
-- `[ ]` — pending
-- `[x]` — completed
-- `[-]` — skipped (with reason)
-- `#in-progress` — currently being worked on (add inline, don't move task)
+- [ ] T1.4.1: Implement search index in `internal/recall/search/`
+  - Inverted index: term → sessionIds
+  - `Build(sessions []*Session)`
+  - Tokenize: lowercase, split whitespace, remove punctuation
+
+- [ ] T1.4.2: Implement search query
+  - `Search(query string, limit int) []string`
+  - AND semantics (all terms must match)
+  - Sort by term frequency score
+
+### Phase 5: CLI
+
+- [ ] T1.5.1: Add recall subcommand
+  - Create `cmd/ctx/recall.go`
+  - `ctx recall serve <path>` — start server
+  - `--port` flag (default: 8080)
+  - `--open` flag to open browser
+
+- [ ] T1.5.2: Add help and validation
+  - Validate path exists and has JSONL files
+  - Print URL when server starts
+  - Print stats (sessions loaded, time taken)
+
+## Backlog
+
+- [x] Rename vars in the config package.
+- [x] Why is agent runbook lowest in reading priority order?
+  - follow-up: is it enforced?
+- [x] Create a list of what CLI options (if any) are not implemented yet.
+- [ ] Verify all Markdown files by "actually reading them"; take notes for
+  follow-up actions.
+- [x] All go code should have godoc and testing.
+- [ ] GitHub CI linter is giving errors that need fixing.
+- [x] Manual code review. take notes.
+- [x] Add tests per file.
+- [ ] validate everything in the docs with a skeptical eye.
+- [x] consider the case where `ctx` is not called from within AI prompt:
+  - does the command still make sense?
+  - does it create the expected output?
+- [ ] Cut the first release.
+  - Versioning strategy.
+  - Always have a `latest` tag pointing to the latest release.
+  - Or, maybe just use the `latest` tag at all times?
+- [ ] have a proper email for security vulnerability reports.
+- [ ] compare versions of recent change and the last AI-assisted version and
+      ask AI what we have learned about this.
+- [ ] CREATE SHORTS and VODS
+- [ ] Trace the entire git history and sessions, create an extensive document
+      of what we did and how it progressed, and then create a blog post about it.
 
-**Archives**: See `.context/archive/` for completed tasks from previous phases.

.context/archive/tasks-2026-01-25.md

@@ -0,0 +1,39 @@
+## In Progress
+
+## Phase 1
+
+- [ ] Add a `--no-integrations` or `--skip-integrations` or `--skip-hooks` or
+  `--no-hooks` flag to `ctx init` so standalone
+  users can skip the Claude Code (and other possible future)
+  integration setup.
+
+## Phase 2
+
+- [ ] Enforce test coverage targets in CI/Makefile #priority:medium #area:quality
+    - internal/cli: 60% (currently 62.8%)
+    - internal/context: 80% (currently 86.8%)
+    - internal/drift: 80% (currently 88.0%)
+    - internal/claude: 80% (currently 87.5%)
+    - internal/templates: 80% (currently 88.9%)
+
+## Completed (Recent)
+
+## Blocked
+
+## Reference
+
+**Specs** (in `specs/` directory):
+- `core-architecture.md` — Overall design philosophy
+- `go-cli-implementation.md` — Go project structure and patterns
+- `cli.md` — All CLI commands and their behavior
+- `context-file-formats.md` — File format specifications
+- `context-loader.md` — Loading and parsing logic
+- `context-updater.md` — Update command handling
+
+**Task Status Labels**:
+- `[ ]` — pending
+- `[x]` — completed
+- `[-]` — skipped (with reason)
+- `#in-progress` — currently being worked on (add inline, don't move task)
+
+**Archives**: See `.context/archive/` for completed tasks from previous phases.