Skip to content

Records Office redesign, backend hardening, and README screenshots - #3

Merged
imthi16 merged 6 commits into
mainfrom
new_feature
Jul 17, 2026
Merged

imthi16 merged 6 commits into
mainfrom
new_feature

Conversation

@imthi16

@imthi16 imthi16 commented Jul 17, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Frontend — "The Records Office" redesign. Replaces the dark glassmorphism theme with a light, subject-grounded identity (Avanam = ஆவணம், Tamil for document/record): record-paper surfaces, ledger-green chrome, Besley/Catamaran/IBM Plex Mono type, and a new ConfidenceSeal component that stamps answers VERIFIED (vermilion) at the critic's 0.7 threshold or PROVISIONAL (brass) below it. The pipeline visualizer is restyled as a routing slip with a visible revision loop. Also: lazy-loaded views, capped per-message agent events, tightened TS types, and mobile fixes (collapsed sidebar default, scrollable hero).
  • Backend hardening. The formatter no longer makes an LLM call (pass-through + citation previews); get_llm() caches ChatOllama instances; GET /api/query/history gains ?limit=; agent-run summaries are truncated before persisting; empty documents get status="empty" and skip FAISS indexing.
  • Docs. README gains a Screenshots section backed by five captures in docs/screenshots/, plus factual fixes (React 19, qwen2.5:7b default model, corrected name etymology). CLAUDE.md refreshed for the current architecture; AGENTS.md contributor guide added.

Screenshots

See the README's new 📸 Screenshots section on this branch — chat hero, live pipeline mid-revision, a stamped answer with citations, the document registry, and the query ledger.

Test plan

  • npm run build (tsc + vite) passes
  • npm run lint passes
  • All three views + mobile width verified visually via headless Chrome against a mocked API (screenshots in docs/screenshots/)
  • End-to-end query against the real stack (docker-compose up) — needs Ollama running

🤖 Generated with Claude Code

mdimthi and others added 4 commits July 2, 2026 17:03
- Formatter no longer makes an LLM call: it passes the analyst's answer
  through and builds truncated citation previews (only analyst and critic
  hit the LLM now)
- Cache ChatOllama instances in get_llm() per (model, base_url,
  temperature, max_tokens)
- Add ?limit= to GET /api/query/history (default 50, max 200) and truncate
  AgentRun output summaries before persisting
- Handle empty documents: files that parse to no text get status="empty"
  and skip FAISS indexing; UTF-8 fast path before chardet fallback

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Replace the dark glassmorphism theme with a light, subject-grounded
identity: Avanam (ஆவணம், Tamil for document/record) styled as a registry.

- New token system in index.css: record paper, ledger green chrome,
  iron ink, vermilion reserved for verification, brass for revisions
- Type: Besley (display), Catamaran (UI, Latin+Tamil), IBM Plex Mono (data)
- New ConfidenceSeal component: SVG stamp on answers — VERIFIED at the
  critic's 0.7 threshold, PROVISIONAL below it
- Pipeline visualizer restyled as a routing slip with a brass revision loop
- Lazy-loaded views, capped per-message agent events, tightened TS types,
  sidebar starts collapsed on narrow viewports, scrollable hero empty state

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- Add docs/screenshots (chat hero, live pipeline with revision loop,
  stamped answer, document registry, query ledger) and a README
  screenshots section
- Correct README facts: Avanam = ஆவணம் (document/record), React 19,
  qwen2.5:7b default model, Records Office design system
- Update CLAUDE.md for the current architecture; add AGENTS.md
  contributor guide

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

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2d7eb89aee

ℹ️ 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".

Comment thread backend/app/agents/router.py Resolved
mdimthi and others added 2 commits July 17, 2026 14:42
- Router: only short-circuit to a canned chat reply when the query
  consists entirely of greeting/chat phrases — mixed queries like
  "Hi, summarize the uploaded contract" now route through retrieval
- DocumentUpload: stop the Browse button's click from bubbling to the
  dropzone root, which opened a second file picker
- Fix ruff findings (unused confidence_score in formatter, unused
  imports) and apply black to the files touched by this PR

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Takes main's hardened backend (thread-safe vector store, cosine scoring,
JSON-mode LLM, retriever short-circuit, Alembic entrypoint, pytest suite)
and keeps this branch's Records Office frontend redesign. Notable
resolutions:

- router: main's classify_intent() structure with the PR-review fix ported
  onto it — chat patterns are now full-match, so mixed queries like
  "hello summarize this doc" route to search (passes main's router tests)
- query route: main's version plus this branch's ?limit= param and
  truncated JSON agent-run summaries; persist real pipeline duration
- pipeline: restore revision_count in the terminal SSE event (the UI's
  "Revised ×N" badge reads it)
- frontend types: merge main's AgentEventData/QueryHistoryItem with this
  branch's agent-name aliases; drop useApi.ts (deleted on main, unused)
- docs: keep main's CLAUDE.md and add a Records Office design-system
  section; README keeps screenshots with main's llama3.2:3b model facts

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@imthi16
imthi16 merged commit 56fa119 into main Jul 17, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants