Feat/intent workspace parser and trace fixes#11661
Closed
MamaMoh wants to merge 11 commits intoRooCodeInc:mainfrom
Closed
Feat/intent workspace parser and trace fixes#11661MamaMoh wants to merge 11 commits intoRooCodeInc:mainfrom
MamaMoh wants to merge 11 commits intoRooCodeInc:mainfrom
Conversation
Comprehensive documentation of Roo Code architecture including: - System overview with Mermaid component diagrams - Component-level architecture and execution flows - Tool execution flows (write_to_file, execute_command) - Message flow sequence diagrams - Hook integration points and implementation roadmap - Critical code references for development
Docs/trp1 architecture documentation
…t tools, hooks, and MCP config - Add GetActiveIntent and SelectActiveIntent native tools and prompt sections - Add intent-governance prompt section and wire into system prompts - Extend BaseTool and Task for intent/hook flow; register commands and tools - Add hooks infra, orchestration and specs dirs, and cursor rules - Add .cursor/mcp.json for tenxfeedbackanalytics and GitHub MCP - Update types (tool, vscode), presentAssistantMessage, and ARCHITECTURE_NOTES - Bump deps and pnpm-lock; add tests dir
feat(001-intent-hook-middleware): add intent governance, active-inten…
- Updated spec.md to mark completed vs remaining user stories - Added implementation status section showing what's done - Updated functional requirements and success criteria with status - Updated plan.md with completed components and remaining work - Documented that User Stories 1, 2, 6 are complete - Identified remaining work: User Stories 3, 4, 5 (trace logging, hashing, optimistic locking)
…n-status docs: update spec and plan with implementation status
…nt Hashing - Implement TraceManager class with mutation_class determination and content hashing - Add PostToolHook to log file write operations to agent_trace.jsonl - Integrate PostToolHook into HookEngine - Add comprehensive unit tests for TraceManager - Update spec and tasks to reflect completion status User Story 3 (Traceability Logging): - TraceManager creates trace entries with intent ID, content hash, file path, mutation class, and timestamp - PostToolHook logs successful destructive tool executions - Non-blocking error handling for trace logging failures User Story 4 (Content Hashing): - SHA256 content hash computation from raw file bytes (no normalization) - Performance target: <50ms for files up to 1MB - Hash integrity preserved for line endings and whitespace differences
…n-status feat: implement User Stories 3 and 4 - Traceability Logging and Conte…
…empty handling - Inject current intent context into system prompt when activeIntentId is set (Task.getSystemPrompt appends formatIntentContext). - Add FileStateLockStore and optimistic locking: record hash on read, block stale writes in PreToolHook, update after write in PostToolHook; wire in extension and ReadFileTool. - Trace write_to_file and edit_file in spec format (operation, content_hash, classification); use pre-write mutation class for classification. - PreToolHook: distinct error when no intents in YAML vs no intent selected.
…tent messaging - NativeToolCallParser: add intent_id to toolParamNames, select_active_intent and get_active_intent cases; NativeToolArgs for both. - ALWAYS_AVAILABLE_TOOLS: include select_active_intent and get_active_intent so they work in all modes. - IntentManager/OrchestrationStorage: load intents from task workspace (loadIntents/getIntent/setActiveIntent accept workspaceRoot); per-workspace cache so INT-001 is found in project's .orchestration/active_intents.yaml. - PostToolHook: trace only on successful write; appendTraceEntry(entry, workspaceRoot) to write to task's .orchestration/agent_trace.jsonl. - GetActiveIntentTool: distinguish no intents in YAML vs no intent selected; use task.workspacePath for loadIntents.
MamaMoh
requested review from
cte,
daniel-lxs,
hannesrudolph,
jr and
mrubens
as code owners
dosubot
bot
added
size:XXL
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
1 participant
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.
Related GitHub Issue
Closes: #
Roo Code Task Context (Optional)
Description
Test Procedure
Pre-Submission Checklist
Screenshots / Videos
Documentation Updates
Additional Notes
Get in Touch
Interactively review PR in Roo Code Cloud