fischer-agentkit

Commit Graph

Author	SHA1	Message	Date
chiguyong	f99b3517d9	fix(review): apply code review fixes from ce-code-review - P1: Use _resolve_execution_mode() instead of hardcoding SKILL_REACT in semantic_low_complexity, semantic_high, and merged_llm paths - P1: QualityGate escalation uses name-based check (c.name) instead of identity check (c is) for robustness - P2: Remove tautological complexity >= 0.3 in short_text_llm_hint - P2: Add empty query guard in SemanticRouter.route() - P2: Upgrade debug → warning log level for low-complexity fallback errors - P2: Validate skill_hint against _SKILL_NAME_RE in _classify_merged - P2: Rename has_high_signal → has_non_low_signal for clarity	2026-06-16 00:24:14 +08:00
chiguyong	11e2009cb8	feat(router): improve colloquial/mixed-lang routing, fix low-complexity IntentRouter bypass Key improvements: - Low-complexity queries (<0.3) now try IntentRouter keyword match before falling back to DIRECT_CHAT, fixing 0% F1 on keyword_match - SemanticRouter similarity_low lowered from 0.6 to 0.4 - Short text (<20 chars) uses effective_low = max(0.25, low - 0.15) - Short text with no semantic match forces LLM classify fallback - Added colloquial keywords to 7 skill YAMLs - Fixed code_reviewer.yaml output_schema placement - Fixed SemanticRouter build in e2e tests - Fixed base_url detection for bailian-coding API keys Results: keyword_match F1 0->60.87%, colloquial F1 0->100%, mixed_lang F1 0->100%	2026-06-15 23:54:57 +08:00
chiguyong	0ccef7be5c	feat: P0 production hardening — LLM cache, semantic routing, state persistence U1: LLM Cache Core (exact + semantic match, InMemory + Redis backends) U2: Cache integration into LLMGateway with CacheConfig U3: Semantic Router as Layer 1.5 in CostAwareRouter U4: UsageStore persistence (Redis Hash + InMemory fallback) U5: CascadeStateStore persistence (Redis INCR + InMemory TTL) U6: EvolutionStore interface unification (Protocol + PostgreSQL backend) U7: Configuration integration + E2E tests Code review fixes: - P0: date iteration bug (day>=28), semantic router index never built, Redis connection leak (per-call → persistent pool) - P1: cache degradation recovery, semantic_search degradation, double miss counting, asyncio.Lock for PG init, LIMIT on queries, __import__ anti-pattern → _utcnow() - P2: InMemory TTL cleanup, embedding preservation on put(), data TTL = max(exact_ttl, semantic_ttl)	2026-06-14 15:16:00 +08:00

Author

SHA1

Message

Date

chiguyong

f99b3517d9

fix(review): apply code review fixes from ce-code-review

- P1: Use _resolve_execution_mode() instead of hardcoding SKILL_REACT
  in semantic_low_complexity, semantic_high, and merged_llm paths
- P1: QualityGate escalation uses name-based check (c.name) instead
  of identity check (c is) for robustness
- P2: Remove tautological complexity >= 0.3 in short_text_llm_hint
- P2: Add empty query guard in SemanticRouter.route()
- P2: Upgrade debug → warning log level for low-complexity fallback errors
- P2: Validate skill_hint against _SKILL_NAME_RE in _classify_merged
- P2: Rename has_high_signal → has_non_low_signal for clarity

2026-06-16 00:24:14 +08:00

chiguyong

11e2009cb8

feat(router): improve colloquial/mixed-lang routing, fix low-complexity IntentRouter bypass

Key improvements:
- Low-complexity queries (<0.3) now try IntentRouter keyword match
  before falling back to DIRECT_CHAT, fixing 0% F1 on keyword_match
- SemanticRouter similarity_low lowered from 0.6 to 0.4
- Short text (<20 chars) uses effective_low = max(0.25, low - 0.15)
- Short text with no semantic match forces LLM classify fallback
- Added colloquial keywords to 7 skill YAMLs
- Fixed code_reviewer.yaml output_schema placement
- Fixed SemanticRouter build in e2e tests
- Fixed base_url detection for bailian-coding API keys

Results: keyword_match F1 0->60.87%, colloquial F1 0->100%, mixed_lang F1 0->100%

2026-06-15 23:54:57 +08:00

chiguyong

0ccef7be5c

feat: P0 production hardening — LLM cache, semantic routing, state persistence

U1: LLM Cache Core (exact + semantic match, InMemory + Redis backends)
U2: Cache integration into LLMGateway with CacheConfig
U3: Semantic Router as Layer 1.5 in CostAwareRouter
U4: UsageStore persistence (Redis Hash + InMemory fallback)
U5: CascadeStateStore persistence (Redis INCR + InMemory TTL)
U6: EvolutionStore interface unification (Protocol + PostgreSQL backend)
U7: Configuration integration + E2E tests

Code review fixes:
- P0: date iteration bug (day>=28), semantic router index never built,
      Redis connection leak (per-call → persistent pool)
- P1: cache degradation recovery, semantic_search degradation,
      double miss counting, asyncio.Lock for PG init, LIMIT on queries,
      __import__ anti-pattern → _utcnow()
- P2: InMemory TTL cleanup, embedding preservation on put(),
      data TTL = max(exact_ttl, semantic_ttl)

2026-06-14 15:16:00 +08:00

3 Commits