tony/CHORUS
1
0
Fork 0
Code Issues 10 Pull Requests Actions Packages Projects Releases Wiki Activity

CHORUS-SLURP-002: Implement Missing SLURP Temporal Graph Core Components #2

New Issue
Open
opened 2025-09-21 06:15:50 +00:00 by tony · 0 comments
tony commented 2025-09-21 06:15:50 +00:00
Owner
Copy Link

Problem

Critical SLURP functionality missing - temporal navigation and decision analysis components are stubbed out, blocking SEC-SLURP 1.1 milestone.

Location

File: pkg/slurp/slurp.go
Lines: 578-594 (GetTemporalEvolution), 612-614 (GenerateContext)

Missing Implementations

1. GetTemporalEvolution() - Lines 578-594

Current: Returns empty array with comment "Temporal analysis will be delivered in later roadmap phases"
Required: Implement full temporal graph analysis

2. NavigateDecisionHops() - Lines 578-594

Current: Returns "not implemented" error
Required: Implement decision record navigation and hop analysis

3. GenerateContext() - Lines 612-614

Current: Returns "not implemented" error
Required: Implement admin-only context generation with proper authorization

Technical Requirements

  1. Temporal Graph Implementation:

    • Track decision record evolution over time
    • Support UCXL temporal operators (~~/, ^^/)
    • Maintain causal relationships between decisions

  2. Decision Navigation:

    • Implement hop-based traversal of decision graphs
    • Support scoped visibility (role-based access)
    • Handle redacted justifications properly

  3. Admin Context Generation:

    • Enforce admin-only access control
    • Generate consolidated context views
    • Support multiple output formats

Priority

High - Blocking SEC-SLURP 1.1 milestone

Requirement Traceability

@goal: CHORUS-SLURP-002, SEC-SLURP-1.1 - Temporal analysis and context generation

Dependencies

  • Requires UCXL temporal operator support
  • Depends on decision record persistence layer
  • Needs integration with BUBBLE for justifications

Acceptance Criteria

  • GetTemporalEvolution() returns real temporal analysis data
  • NavigateDecisionHops() implements working graph traversal
  • GenerateContext() enforces admin permissions and returns valid context
  • All methods include proper error handling and logging
  • Unit tests cover all new functionality
  • Integration tests verify UCXL temporal operator compatibility
## Problem Critical SLURP functionality missing - temporal navigation and decision analysis components are stubbed out, blocking SEC-SLURP 1.1 milestone. ## Location **File:** `pkg/slurp/slurp.go` **Lines:** 578-594 (GetTemporalEvolution), 612-614 (GenerateContext) ## Missing Implementations ### 1. GetTemporalEvolution() - Lines 578-594 **Current:** Returns empty array with comment "Temporal analysis will be delivered in later roadmap phases" **Required:** Implement full temporal graph analysis ### 2. NavigateDecisionHops() - Lines 578-594 **Current:** Returns "not implemented" error **Required:** Implement decision record navigation and hop analysis ### 3. GenerateContext() - Lines 612-614 **Current:** Returns "not implemented" error **Required:** Implement admin-only context generation with proper authorization ## Technical Requirements 1. **Temporal Graph Implementation:** - Track decision record evolution over time - Support UCXL temporal operators (~~/, ^^/) - Maintain causal relationships between decisions 2. **Decision Navigation:** - Implement hop-based traversal of decision graphs - Support scoped visibility (role-based access) - Handle redacted justifications properly 3. **Admin Context Generation:** - Enforce admin-only access control - Generate consolidated context views - Support multiple output formats ## Priority **High** - Blocking SEC-SLURP 1.1 milestone ## Requirement Traceability @goal: CHORUS-SLURP-002, SEC-SLURP-1.1 - Temporal analysis and context generation ## Dependencies - Requires UCXL temporal operator support - Depends on decision record persistence layer - Needs integration with BUBBLE for justifications ## Acceptance Criteria - [ ] GetTemporalEvolution() returns real temporal analysis data - [ ] NavigateDecisionHops() implements working graph traversal - [ ] GenerateContext() enforces admin permissions and returns valid context - [ ] All methods include proper error handling and logging - [ ] Unit tests cover all new functionality - [ ] Integration tests verify UCXL temporal operator compatibility
tony added the
bzzz-task
 label 2025-09-21 06:32:43 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
Something is not working
bzzz-task
CHORUS task for auto ingestion.
chorus-entrypoint
Marks issues that trigger council formation for project kickoffs
duplicate
This issue or pull request already exists
enhancement
New feature
help wanted
Need some help
invalid
Something is wrong
question
More information is needed
wontfix
This won't be fixed
No Label
bzzz-task
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
tony
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: tony/CHORUS#2
No description provided.
Delete Branch "%!s()"

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?