V3#101
Merged
Merged
Conversation
Signed-off-by: isolomatov-gd <isolomatov@griddynamics.com>
Update CODEOWNERS, fix CI setup, and enhance plugin functionality
Signed-off-by: isolomatov-gd <isolomatov@griddynamics.com>
Signed-off-by: isolomatov-gd <isolomatov@griddynamics.com>
Signed-off-by: isolomatov-gd <isolomatov@griddynamics.com>
… contract Signed-off-by: isolomatov-gd <isolomatov@griddynamics.com>
Signed-off-by: isolomatov-gd <isolomatov@griddynamics.com>
Signed-off-by: isolomatov-gd <isolomatov@griddynamics.com>
Signed-off-by: isolomatov-gd <isolomatov@griddynamics.com>
Signed-off-by: isolomatov-gd <isolomatov@griddynamics.com>
Previously fired on Write|Edit (and Copilot replace_string_in_file, multi_replace_string_in_file, and Codex apply_patch Update File), causing noise and spawn overhead on every in-place edit. Now fires only on actual creation: Write/create_file across all IDEs, and apply_patch Add File/Create File for Codex. Two-layer fix: JSON matchers prevent the process from spawning on edit tools entirely; PATCH_FILE_RE tightened from Update|Add|Create to Add|Create so Codex update patches return early at the code level. Also fixes pre-existing mypy errors in analytics/tracker.py (str cast on header dict values that typed as Any). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…te tests accordingly
…te IDE detection logic
- New hook: md-file-advisory.ts with tests and per-IDE bundles - Refactor build-bundles.mjs to iterate HOOK_SOURCES array instead of hardcoding a single entry - Register md-file-advisory.js in hooks.json configs for Claude, Codex, Copilot, Cursor - Document HOOK_SOURCES registration step in ARCHITECTURE.md - Add .codex to .gitignore Made-with: Cursor
Plan A.1 (Copilot CLI runtime trace) was not executed by either side, so the team lead's d2a94c6 ("Copilot plugin works too") is treated as authoritative. The shape inversion in 82fb05e/c612baf is reverted: before: out.additionalContext = additionalContext (flat) after: out.hookSpecificOutput = { hookEventName, additionalContext } (nested) Surgical revert — only the shape line in formatOutput and corresponding test assertions. Dual-IDE routing in adapter-copilot.ts, Cursor test expansion, claude-plugin-root smoke test, and file-creation narrowing (49017c5) are all preserved. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: isolomatov-gd <isolomatov@griddynamics.com>
Signed-off-by: isolomatov-gd <isolomatov@griddynamics.com>
Signed-off-by: isolomatov-gd <isolomatov@griddynamics.com>
Signed-off-by: isolomatov-gd <isolomatov@griddynamics.com>
Signed-off-by: isolomatov-gd <isolomatov@griddynamics.com>
Made-with: Cursor
Remove md-file-advisory.js from .github/plugin/ (not needed there) and add the md-file-advisory hook entry to PostToolUse in both hooks.json and hooks.json.tmpl so the hook is properly invoked at runtime. Made-with: Cursor
Add md-file-advisory.js entry to PostToolUse in both hooks.json and hooks.json.tmpl under plugins/core-copilot/hooks/. Made-with: Cursor
Add md-file-advisory.js entry to postToolUse in both hooks.json and hooks.json.tmpl under plugins/core-cursor/hooks/. Made-with: Cursor
…oncatenation, inherit stdio for log output
Gate md-file-advisory on PostToolUse event and file-write tools only (Write, Edit, apply_patch, create_file, etc.) to prevent noise from Read, Bash, Search and other non-mutating tools. Extract getFilePath() helper with file_path/filePath/path priority chain and apply_patch command parsing. Add 29 new tests covering shouldCheck, getFilePath, and integration scenarios for filtered tools. Addresses PR #75 review comment from isolomatov-gd. Made-with: Cursor
Remove stale .codex ignore rule and trailing-newline fix in .gitignore. Add md-file-advisory.js hook entry to core-copilot/hooks.json PostToolUse. Made-with: Cursor
…anager is unavailable
…agent, clean core concepts
…hemas, and eliminating duplications (subagents_orchestration_rules)
…an_manager-from-bootstrap-and-MCP-instructions-v3 Introduce plan manager to bootstrap files
Add GitNexus integration test scenarios for CTORNDGAIN-1494. TestRail-compatible cases covering GitNexus MCP validation (20 scenarios). Co-authored-by: Cursor <cursoragent@cursor.com>
* test(hooks): per-IDE toMatchObject assertions + offline QA evidence (B4/B5/Mi1 fixes) * qa(hooks): gap-issues filed, handoff doc, partial QA report (Tasks 8-9 partial) Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
isolomatov-gd
requested review from
ElizaVetaFomka,
YevheniiaLementova,
kkhristenko51 and
omaiesh
as code owners
Signed-off-by: isolomatov-gd <isolomatov@griddynamics.com>
Signed-off-by: isolomatov-gd <isolomatov@griddynamics.com>
Signed-off-by: isolomatov-gd <isolomatov@griddynamics.com>
Contributor
📋 Prompt Quality Validation Report✅ Validation PassedSummary by File
|
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
4 participants
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.
Add R3 with all changes to everything in Rosetta