docs: add CLAUDE.md, architecture docs, and /sesh-mode skill#6247
Merged
docs: add CLAUDE.md, architecture docs, and /sesh-mode skill#6247
Conversation
g-talbot
force-pushed
the
gtt/docs-claude-md
branch
from
0ee18f5 to
b077bf3
Compare
g-talbot
force-pushed
the
gtt/phase-31-execute
branch
from
ae409ac to
3eb3316
Compare
g-talbot
force-pushed
the
gtt/docs-claude-md
branch
from
70a48be to
d225cc7
Compare
g-talbot
force-pushed
the
gtt/phase-31-execute
branch
2 times, most recently
from
c4418dd to
aa5ff45
Compare
g-talbot
force-pushed
the
gtt/docs-claude-md
branch
2 times, most recently
from
8d701de to
bcdda05
Compare
g-talbot
force-pushed
the
gtt/phase-31-execute
branch
2 times, most recently
from
142ddd6 to
972bd20
Compare
g-talbot
force-pushed
the
gtt/docs-claude-md
branch
from
bcdda05 to
0fd7087
Compare
g-talbot
force-pushed
the
gtt/phase-31-execute
branch
2 times, most recently
from
73c90c4 to
44bd6bb
Compare
g-talbot
force-pushed
the
gtt/docs-claude-md
branch
from
468ac45 to
1e5722d
Compare
g-talbot
force-pushed
the
gtt/phase-31-execute
branch
from
44bd6bb to
add1902
Compare
g-talbot
force-pushed
the
gtt/docs-claude-md
branch
from
1e5722d to
9b6c961
Compare
g-talbot
force-pushed
the
gtt/phase-31-execute
branch
from
add1902 to
d88f67f
Compare
g-talbot
force-pushed
the
gtt/docs-claude-md
branch
2 times, most recently
from
06a6296 to
8f1cbde
Compare
- Resolve Cargo.lock/Cargo.toml merge conflicts - P1 (sort column lookup): Already addressed by sort fields tag_ prefix fix — sort field names now match Parquet column names - P2 (window_start at epoch 0): Remove time_range.start_secs > 0 guard so window_start is computed for all batches when window_duration > 0 Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
- Resolve writer.rs conflict: keep META-07 self-describing roundtrip test - P1 (create_timestamp serde): Add #[serde(default)] to StoredMetricsSplit.create_timestamp for backward-compatible reads of pre-existing file-backed index JSON - P1 (compaction window overlap): No change needed — Bound::Included vs Bound::Excluded already handles half-open interval semantics correctly, and the edge case (zero duration) is impossible - fields.rs: No change — Matt noted it resolves with wide schema rebase Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
- Resolve postgres.rs conflict: keep check_invariant! macros, add window_duration_secs consistency check - Group setup_dogstatsd_exporter + setup_invariant_recorder into single setup_metrics() function (fulmicoton-dd review) - Rename `id` to `invariant_id` in invariant_recorder (fulmicoton-dd review) Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
2 tasks
Reverts c8bf8d7, cafcac5, a088f53 — these are code changes (delete_metrics_splits error handling, doc comment tweaks) that don't belong in a docs-only PR. They will land in a separate PR. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
This ADR contains company-specific information and should live in the private fork, not in the upstream quickwit-oss repo. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
- Rewrite CLAUDE.md as generic Quickwit AI development guide - Replace Quickhouse-Pomsky -> Quickwit branding across all docs - Replace "Datadog" observability references with generic "production observability" language - Remove "Husky (Datadog)" qualifier from gap docs (keep Husky citations — the blog post is public) - Generalize internal knowledge (query rate numbers, product-specific lateness guarantees) - Remove PomChi reference, private Google Doc link - Add docs/internals/UPSTREAM-CANDIDATES.md for pomsky tracking Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
- Remove all ClickHouse/ClickStack references from gap docs and ADRs (keep Prometheus, Mimir, InfluxDB, Husky as prior art) - Restore gap-005 Option C (compaction-time dedup) without ClickHouse citation - Mark /sesh-mode reference in CLAUDE.md as aspirational - Add aspirational items section to UPSTREAM-CANDIDATES.md tracking items described in docs but not yet implemented (TLA+ specs, DST, Kani, Bloodhound, performance baselines, benchmark binaries) Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
UPSTREAM-CANDIDATES.md incorrectly stated TLA+ specs and Stateright models don't exist. They do (contributed in #6246): ParquetDataModel.tla, SortSchema.tla, TimeWindowedCompaction.tla, plus quickwit-dst invariants and Stateright model tests. Updated to accurately reflect that the remaining aspirational piece is the simulation infrastructure (SimClock, FaultInjector, etc.). Also removed the /sesh-mode aspirational entry — it's actively being used and the underlying specs/models are real. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Prevents GSD planning artifacts from being committed to the repository. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Reverts test env vars (CP_ENABLE_REVERSE_CONNECTION) and load-cloudprem-ui target — these are pomsky-specific and don't belong in upstream. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
g-talbot
force-pushed
the
gtt/docs-claude-md
branch
from
cc4492e to
4006b20
Compare
Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
alanfgates
approved these changes
Apr 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/sesh-modeskill for verification-first workflow (TLA+, DST, Stateright)Combines pomsky PRs #468 and #474.
Stacks on
gtt/phase-31-execute(PR #6246).The split logic (from PR #474)
/sesh-modeTest plan
/sesh-modeskill contains the full verification workflow🤖 Generated with Claude Code