[UPDATE PRIMITIVE] Fix CallGraph docs: remove IDE integration claim, fix output format, fix NON_COMPLIANT typo

[Copilot]

Three documentation correctness fixes across all 16 CallGraph{From,To}.md files (8 languages) and the shared instructions file, addressing review feedback on #81.

📝 Update Information

Primitive Details

• Type: Documentation
• Name: CallGraphFrom, CallGraphTo (all languages: cpp, csharp, go, java, javascript, python, ruby, swift)
• Update Category: Bug Fix (incorrect documentation claims)

⚠️ CRITICAL: PR SCOPE VALIDATION

✅ ALLOWED FILES:

• Documentation updates (server/ql/*/tools/src/CallGraph{From,To}/*.md)
• Instructions file (.github/instructions/server_ql_languages_tools.instructions.md)

---

🛑 MANDATORY PR VALIDATION CHECKLIST

• ONLY server implementation files are included
• NO temporary or output files are included
• NO unrelated configuration files are included
• ALL existing tests continue to pass
• NEW functionality is properly tested

---

• Impact Scope: Localized — documentation only, no behavioral changes

Update Metadata

• Breaking Changes: No
• API Compatibility: Maintained
• Performance Impact: Neutral

🎯 Changes Description

Current Behavior

All CallGraph{From,To}.md files incorrectly listed "IDE integration for call hierarchy navigation" as a use case. The example message patterns and Output Format sections used single-quoted placeholders ('source', 'callee') that don't match the actual query output, which uses backtick-quoted names. The instructions file had a NON-COMPLIANT hyphen typo (should be underscore, consistent with NON_COMPLIANT used elsewhere).

Updated Behavior

• Use Cases: Removed the false "IDE integration for call hierarchy navigation" bullet from all 16 files.
• Output format: Updated example message patterns and Output Format sections to use double-backtick spans reflecting actual query output:

<!-- BEFORE -->
Running with `sourceFunction = "sourceFunc"` produces results ... with the message pattern `Call from 'sourceFunc' to 'helper1'`.

- `select call, "Call from 'source' to 'callee'"`

<!-- AFTER -->
Running with `sourceFunction = "sourceFunc"` produces results ... with the message pattern ``Call from `sourceFunc` to `helper1``.

- ``select call, "Call from `source` to `callee`"``

• Instructions typo: NON-COMPLIANT → NON_COMPLIANT in .github/instructions/server_ql_languages_tools.instructions.md.

Motivation

The IDE integration claim was inaccurate — these queries are standalone call-graph analysis tools, not IDE call hierarchy providers. The single-quoted output format conflicted with what the .ql files actually produce (backtick-wrapped names via string concatenation), causing user confusion when comparing docs to actual results.

🔄 Before vs. After Comparison

Functionality Changes

No functional changes — documentation corrections only.

API Changes

None.

Output Format Changes

No change to actual query output. Documentation now accurately reflects the existing backtick format.

🧪 Testing & Validation

Test Coverage Updates

• Existing Tests: All existing tests continue to pass
• New Test Cases: N/A — documentation-only change
• Regression Tests: N/A
• Edge Case Tests: N/A

Validation Scenarios

1. Backward Compatibility: No code changes; fully compatible.
2. New Functionality: N/A.
3. Error Handling: N/A.
4. Performance: N/A.

Test Results

• Unit Tests: All pass (910/910)
• Integration Tests: Pre-existing unrelated failure in codeql_pack_install/install_pack
• Manual Testing: Verified all 16 files have no remaining single-quoted patterns or IDE integration claims

📋 Implementation Details

Files Modified

• Documentation: server/ql/{cpp,csharp,go,java,javascript,python,ruby,swift}/tools/src/CallGraph{From,To}/CallGraph{From,To}.md (16 files)
• Documentation: .github/instructions/server_ql_languages_tools.instructions.md

Code Changes Summary

• Documentation: Removed false IDE integration use case, corrected output format backticks, fixed NON_COMPLIANT typo

Dependencies

• No New Dependencies: Documentation-only changes

🔍 Quality Improvements

Bug Fixes (if applicable)

• Issue: CallGraph docs claimed IDE integration support; output format used wrong quote style
• Root Cause: Copy-paste error when authoring the initial .md files
• Solution: Removed the incorrect use case bullet; updated format strings to match actual QL select string interpolation
• Prevention: N/A — enforced by human review

Code Quality Enhancements

• Readability: Docs now accurately reflect query behavior

🔗 References

Related Issues/PRs

• Related PRs: Prep for v2.24.2 release #81

🚀 Compatibility & Migration

Backward Compatibility

• Fully Compatible: No breaking changes

API Evolution

• Maintained Contracts: Core API contracts preserved

👥 Review Guidelines

For Reviewers

Please verify:

• ⚠️ SCOPE COMPLIANCE: PR contains only documentation files
• ⚠️ NO UNRELATED FILES: No temporary, output, or unrelated files
• ⚠️ BACKWARD COMPATIBILITY: No functional changes
• Documentation: Corrected claims match actual query behavior

Testing Instructions

npm run build-and-test

📊 Impact Assessment

Server Impact

• Startup Time: No impact
• Runtime Stability: No impact
• Resource Usage: No impact
• Concurrent Usage: No impact

AI Assistant Impact

• Enhanced Accuracy: Docs now correctly describe query output format, reducing LLM confusion when interpreting results

🔄 Deployment Strategy

Rollout Considerations

• Safe Deployment: Documentation-only, safe to deploy immediately

---

Update Methodology: This update follows best practices:

1. ✅ Comprehensive backward compatibility analysis
2. ✅ Thorough testing of all changes
3. ✅ Performance impact assessment
4. ✅ Clear documentation of changes
5. ✅ Robust error handling improvements
6. ✅ Maintained code quality standards

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.