Skip to content

Comments

Trp1 week1 architecture#11659

Open
sumeyaaaa wants to merge 41 commits intoRooCodeInc:mainfrom
sumeyaaaa:trp1-week1-architecture
Open

Trp1 week1 architecture#11659
sumeyaaaa wants to merge 41 commits intoRooCodeInc:mainfrom
sumeyaaaa:trp1-week1-architecture

Conversation

@sumeyaaaa
Copy link

@sumeyaaaa sumeyaaaa commented Feb 21, 2026

Related GitHub Issue

Closes: #

Roo Code Task Context (Optional)

Description

Test Procedure

Pre-Submission Checklist

  • Issue Linked: This PR is linked to an approved GitHub Issue (see "Related GitHub Issue" above).
  • Scope: My changes are focused on the linked issue (one major feature/fix per PR).
  • Self-Review: I have performed a thorough self-review of my code.
  • Testing: New and/or updated tests have been added to cover my changes (if applicable).
  • Documentation Impact: I have considered if my changes require documentation updates (see "Documentation Updates" section below).
  • Contribution Guidelines: I have read and agree to the Contributor Guidelines.

Screenshots / Videos

Documentation Updates

Additional Notes

Get in Touch

Interactively review PR in Roo Code Cloud

Documents the codebase structure and hook system injection points
for the Intent-Code Traceability implementation.
- Add HookEngine middleware with pre/post-hooks for tool governance
- Implement OrchestrationDataModel for .orchestration/ directory management
- Create select_active_intent tool enforcing Reasoning Loop protocol
- Integrate hooks into presentAssistantMessage for all destructive tools
- Add UI-blocking authorization (HITL) for intent evolution
- Implement scope enforcement and trace logging with content hashing
…quirements

- Move HookEngine.ts from src/core/hooks/ to src/hooks/
- Move OrchestrationDataModel.ts from src/core/orchestration/ to src/hooks/
- Move SelectActiveIntentTool.ts from src/core/tools/ to src/hooks/
- Update all import paths in presentAssistantMessage.ts and test files
- Consolidate all hook-related files in clean src/hooks/ directory as required
@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. Documentation Improvements or additions to documentation labels Feb 21, 2026
const normalizedPattern = pattern.replace(/\\/g, "/")

// Convert glob pattern to regex
const regexPattern = normalizedPattern.replace(/\*\*/g, ".*").replace(/\*/g, "[^/]*").replace(/\//g, "\\/")

Check failure

Code scanning / CodeQL

Incomplete string escaping or encoding High

This does not escape backslash characters in the input.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation Improvements or additions to documentation size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant