Rockachopa/Timmy-time-dashboard
1
0
Fork 2
Code Issues 7 Pull Requests Actions Packages Projects Releases Wiki Activity

[claude] Add unit tests for kimi_delegation.py (#1295) #1303

Merged
claude merged 1 commits from claude/issue-1295 into main 2026-03-24 01:54:45 +00:00
Conversation 0 Commits 1 Files Changed 1 +643
claude commented 2026-03-24 01:54:20 +00:00
Collaborator
Copy Link

Fixes #1295

Adds tests/timmy/test_kimi_delegation.py with 40 unit tests covering all public and private functions in src/timmy/kimi_delegation.py:

  • exceeds_local_capacity: keyword detection, word-count threshold, edge cases
  • _slugify: special chars, truncation, whitespace
  • _build_research_template: output structure, priority, label inclusion
  • _extract_action_items: checkboxes, numbered lists, Action/TODO/Next-step prefixes, deduplication
  • _get_or_create_label: existing label lookup, creation, error degradation
  • create_kimi_research_issue: disabled config, success path, API errors, exceptions
  • poll_kimi_issue: disabled config, closed issue detection, timeout handling
  • index_kimi_artifact: empty content guard, success, exception fallback
  • extract_and_create_followups: no items, unconfigured Gitea, multi-issue creation
  • delegate_research_to_kimi: empty task/question validation, delegation forwarding

All 570 unit tests pass.

Fixes #1295 Adds `tests/timmy/test_kimi_delegation.py` with 40 unit tests covering all public and private functions in `src/timmy/kimi_delegation.py`: - `exceeds_local_capacity`: keyword detection, word-count threshold, edge cases - `_slugify`: special chars, truncation, whitespace - `_build_research_template`: output structure, priority, label inclusion - `_extract_action_items`: checkboxes, numbered lists, Action/TODO/Next-step prefixes, deduplication - `_get_or_create_label`: existing label lookup, creation, error degradation - `create_kimi_research_issue`: disabled config, success path, API errors, exceptions - `poll_kimi_issue`: disabled config, closed issue detection, timeout handling - `index_kimi_artifact`: empty content guard, success, exception fallback - `extract_and_create_followups`: no items, unconfigured Gitea, multi-issue creation - `delegate_research_to_kimi`: empty task/question validation, delegation forwarding All 570 unit tests pass.
claude added 1 commit 2026-03-24 01:54:21 +00:00
test: add unit tests for kimi_delegation.py
Some checks failed
Tests / lint (pull_request) Failing after 30s
Details
Tests / test (pull_request) Has been skipped
Details
064f6e0c1d 
Covers all public and private functions:
- exceeds_local_capacity: keyword detection, word count threshold, edge cases
- _slugify: special chars, truncation, whitespace handling
- _build_research_template: output structure, priority, KIMI_READY_LABEL
- _extract_action_items: checkboxes, numbered lists, Action/TODO/Next step prefixes, dedup
- _get_or_create_label: existing label lookup, label creation, error degradation
- create_kimi_research_issue: disabled config, success, API errors, exceptions
- poll_kimi_issue: disabled config, closed issue, timeout
- index_kimi_artifact: empty content, success, exception fallback
- extract_and_create_followups: no items, unconfigured Gitea, multi-issue creation
- delegate_research_to_kimi: missing task/question validation, delegation to create_kimi_research_issue

Fixes #1295

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
claude merged commit bde7232ece into main 2026-03-24 01:54:45 +00:00
claude deleted branch claude/issue-1295 2026-03-24 01:54:45 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
222-epic
Workshop: Timmy as Presence (Epic #222)
 actionable
Has a concrete code/config task extracted
 assigned-claude
Issue currently assigned to Claude agent — do not assign to another agent
 assigned-gemini
Issue currently assigned to Gemini agent — do not assign to another agent
 assigned-groq
assigned-kimi
Issue currently assigned to Kimi agent — do not assign to another agent
 assigned-manus
Issue currently assigned to Manus agent — do not assign to another agent
 claude-ready
consolidation
Part of a consolidation epic
 deprioritized
Keep open but not blocking P0 work
 deprioritized
Keep open but not blocking P0 work
 duplicate
Duplicate of another issue
 gemini-review
Auto-generated by Gemini, needs relevance review
 groq-ready
harness
Core product: agent framework, heartbeat, inference, memory
 heartbeat
Harness: Agent heartbeat loop
 inference
Harness: Inference and model routing
 infrastructure
Supporting stage: dashboard, CI/CD, deployment, DNS
 kimi-ready
Scoped and ready for Kimi to pick up
 memory-session
Harness: Memory and session crystallization
 morrowind
Harness: Morrowind embodiment
 needs-design
Needs architectural design before implementation
 needs-extraction
Philosophy with unextracted engineering work
 p0-critical
Priority 0: Must fix now
 p1-important
Priority 1: Important, next sprint
 p2-backlog
Priority 2: Backlog, do when time permits
 philosophy
Philosophical foundation — informs architecture decisions
 rejected-direction
Closed: rejected or superseded direction
 seed:know-purpose
Three Seeds: KNOW YOUR PURPOSE
 seed:serve-real
Three Seeds: SERVE THE REAL
 seed:tell-truth
Three Seeds: TELL THE TRUTH
 sovereignty
Harness: Sovereignty stack
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Timmy claude grok groq kimi manus perplexity Rockachopa
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Rockachopa/Timmy-time-dashboard#1303
Delete Branch "claude/issue-1295"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?