feat: AI-friendly structured workspace for Claude Code / Codex by STRRL · Pull Request #18 · STRRL/lapp

STRRL · 2026-03-05T08:06:14Z

Summary

Replace all old CLI commands (analyze, debug *) with a new workspace command group: create, add-log, analyze
Build structured file-based workspaces with pattern directories named by LLM-generated semantic IDs — navigable by any AI coding agent or human using basic unix tools
New pkg/workspace/ package with embedded Go templates for AGENTS.md, pattern.md, summary.md, errors.md
Refactor pkg/analyzer/ to expose RunAgentWithPrompt() and BuildWorkspaceSystemPrompt() for the new workspace layout
Remove DuckDB dependency from CLI (--db flag removed); pkg/store/ kept for tests/future use

New CLI

lapp workspace create <dir>
lapp workspace add-log <dir> <logfile> [--model] [--stdin]
lapp workspace analyze <dir> [question] [--model]

Workspace structure

my-investigation/
├── AGENTS.md
├── logs/
├── patterns/
│   ├── server-startup/
│   │   ├── pattern.md
│   │   └── samples.log
│   ├── connection-timeout/
│   │   ├── pattern.md
│   │   └── samples.log
│   └── unmatched/
│       └── samples.log
└── notes/
    ├── summary.md
    └── errors.md

Closes #17

Test plan

make ci passes (fmt, vet, build, lint, unit tests)
lapp workspace create /tmp/test-ws creates directory structure with AGENTS.md
lapp workspace add-log /tmp/test-ws <logfile> populates patterns/ and notes/
echo "test" | lapp workspace add-log /tmp/test-ws --stdin reads from stdin
lapp workspace analyze /tmp/test-ws "what errors?" runs agent

…gest under debug - `analyze` now runs the full ingest pipeline (Drain + semantic labeling + DuckDB storage) before launching the AI agent - Move `ingest` command under `debug ingest` for step-by-step debugging - Extract shared pipeline helpers into `cmd/lapp/pipeline.go` - Remove top-level `templates` command - Add workspace path constraint to analyzer system prompt to prevent the agent from scanning files outside the workspace directory - Add Langfuse tracing support with docker-compose for local dev - Update CLAUDE.md with new CLI structure and code style notes

Instrument the entire pipeline with OTel spans: CLI commands, multiline merge, Drain parsing, semantic labeling, DuckDB storage, and analyzer. HTTP clients for LLM calls use otelhttp transport for deep request traces. - Add pkg/tracing/otel.go with OTLP HTTP exporter (env-gated via OTEL_TRACING_ENABLED) - Add Jaeger service to docker-compose.yml (UI on port 16686) - Wire InitOTel in main.go with graceful shutdown - Add ctx parameter to DrainParser.Feed/Templates and multiline.Merge/MergeSlice - Wrap eino OpenRouter HTTP clients with otelhttp.NewTransport

Extract AnalyzeWithTemplates() that accepts pre-computed templates, so the analyze command passes the same DrainParser output to both DuckDB storage and the workspace builder. Previously, Analyze() created a second DrainParser with fresh UUIDs, causing template IDs in the workspace to diverge from those in the database.

… agents Replace all old CLI commands (analyze, debug *) with a new `workspace` command group (create, add-log, analyze) that builds a structured directory with pattern directories named by LLM-generated semantic IDs. Workspace structure: logs/, patterns/<semantic-id>/{pattern.md,samples.log}, patterns/unmatched/, notes/{summary.md,errors.md}, and AGENTS.md. Closes #17

Keep workspace-based CLI from our branch, discard old analyze/debug/ingest commands and pipeline.go that master modified. Preserve OTel tracing from both branches.

…que stdin names - Sanitize semantic IDs with [a-z0-9-] regex before using as directory names to prevent path traversal from LLM output - Sort filenames before iterating in mergeAllLogs for deterministic rebuild output across runs - Use UnixNano instead of Unix for stdin log filenames to avoid collisions within the same second

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f05c01eb0d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

chatgpt-codex-connector · 2026-03-05T08:34:10Z

pkg/workspace/builder.go

+		label, hasLabel := labelMap[tid]
+		if !hasLabel {
+			continue


Preserve templates when semantic labels are missing

computePatterns drops any Drain template not present in the LLM label map (if !hasLabel { continue }), but semantic.parseResponse accepts partial arrays and does not verify one label per input pattern. When the model omits one pattern_id, all lines for that template are later treated as unmatched, so patterns/ and notes/summary.md lose real clusters and counts become incorrect. Keep unlabeled templates with a fallback semantic ID/description instead of skipping them.

Useful? React with 👍 / 👎.

chatgpt-codex-connector · 2026-03-05T08:34:10Z

cmd/lapp/workspace.go

+	dest := filepath.Join(dir, "logs", filepath.Base(logFile))
+	if err := os.WriteFile(dest, data, 0o644); err != nil {


Avoid overwriting existing logs with same basename

add-log always writes to logs/<basename> with os.WriteFile, so adding two different files that share the same filename (for example /var/log/app.log then /tmp/archive/app.log) silently replaces the first file. Because rebuilds process whatever is in logs/, this can drop previously ingested data and change pattern discovery unexpectedly. Use a collision-safe destination name (or fail on conflict) instead of unconditional overwrite.

Useful? React with 👍 / 👎.

…workspace paths Workspace commands now accept a --topic flag instead of a raw directory path. Topics are sanitized to lower-kebab-case and resolved to ~/.lapp/workspaces/<topic>/, giving users a simpler CLI interface without needing to manage paths directly.

Add `workspace list` subcommand to enumerate existing workspaces. Include available workspace names in error messages when a workspace is not found, helping users discover valid --topic values.

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3f9ad11a9f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

chatgpt-codex-connector · 2026-03-10T03:37:17Z

cmd/lapp/workspace.go

+	if err := copyLogToWorkspace(dir, args, span); err != nil {
+		return err
+	}


Stage new logs until rebuild succeeds

add-log persists the new file into logs/ before running Drain/labeling, but it only regenerates patterns/ and notes/ afterward. If any later step fails (for example semantic.Label returns an error), the command exits with an error while leaving the new log on disk and derived artifacts stale, so subsequent workspace analyze can read summaries that no longer match logs/. This should be made atomic (or explicitly mark the workspace dirty) so failed rebuilds cannot leave inconsistent state.

Useful? React with 👍 / 👎.

STRRL added 4 commits February 27, 2026 20:03

This comment was marked as resolved.

Sign in to view

STRRL added 2 commits March 5, 2026 00:26

merge: resolve conflicts with master (PR #15 restructure CLI)

c013e5f

Keep workspace-based CLI from our branch, discard old analyze/debug/ingest commands and pipeline.go that master modified. Preserve OTel tracing from both branches.

chatgpt-codex-connector bot reviewed Mar 5, 2026

View reviewed changes

STRRL added 2 commits March 9, 2026 20:18

feat: add workspace list command and show available workspaces on errors

3f9ad11

Add `workspace list` subcommand to enumerate existing workspaces. Include available workspace names in error messages when a workspace is not found, helping users discover valid --topic values.

STRRL merged commit ab74f94 into master Mar 10, 2026
2 checks passed

STRRL deleted the spawn-pagent-workspace branch March 10, 2026 03:29

chatgpt-codex-connector bot reviewed Mar 10, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: AI-friendly structured workspace for Claude Code / Codex#18

feat: AI-friendly structured workspace for Claude Code / Codex#18
STRRL merged 8 commits intomasterfrom
spawn-pagent-workspace

STRRL commented Mar 5, 2026

Uh oh!

This comment was marked as resolved.

Uh oh!

chatgpt-codex-connector bot left a comment

Uh oh!

chatgpt-codex-connector bot Mar 5, 2026

Uh oh!

chatgpt-codex-connector bot Mar 5, 2026

Uh oh!

Uh oh!

chatgpt-codex-connector bot left a comment

Uh oh!

chatgpt-codex-connector bot Mar 10, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

		dest := filepath.Join(dir, "logs", filepath.Base(logFile))
		if err := os.WriteFile(dest, data, 0o644); err != nil {

Conversation

STRRL commented Mar 5, 2026

Summary

New CLI

Workspace structure

Test plan

Uh oh!

This comment was marked as resolved.

Uh oh!

chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

chatgpt-codex-connector bot Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

chatgpt-codex-connector bot Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

chatgpt-codex-connector bot Mar 10, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant