Skip to content

Add Claude Code Framework for omnichannel-sdk#183

Merged
bghcore merged 3 commits intomicrosoft:mainfrom
bghcore:main
Feb 12, 2026
Merged

Add Claude Code Framework for omnichannel-sdk#183
bghcore merged 3 commits intomicrosoft:mainfrom
bghcore:main

Conversation

@bghcore
Copy link
Contributor

@bghcore bghcore commented Feb 12, 2026

Add Claude Code Framework
🎯 What
Claude Code instructions (CLAUDE.md) and supporting documentation to standardize AI-assisted development.

🚀 Why
Faster onboarding: New team members (human or AI) can understand architecture instantly
Consistent code quality: AI follows team standards (async/await, error handling, testing)
Cross-repo awareness: AI identifies breaking changes before implementation
Safety: ASK FIRST protocol prevents unauthorized code modifications
📦 What's Included
CLAUDE.md - Entry point linking to existing Copilot instructions + memory banks
✅ Testing
Validated with Claude Code using these scenarios:

✅ "Add a new field to ConversationStarted event" → AI identified all affected products
✅ "Fix a bug in file upload" → AI followed correct coding standards
✅ "Update chat SDK API" → AI recognized public contract implications
🔒 Non-Intrusive
These files are ONLY read by AI tools (Claude Code, GitHub Copilot, etc.)
No build process changes
No runtime impact
Complements (doesn't replace) existing .github/copilot-instructions.md
👀 Review Focus
Please review for:

Accuracy: Are the integration points correct?
Build commands: Do the npm/msbuild/dotnet commands match your workflow?
Standards: Do the coding standards match team practices?

🔗 Related Work
This is part of a coordinated rollout across all Omnichannel repos:

LiveChatWidget - https://github.com/microsoft/CRM.OmniChannel.LiveChatWidget/pull/102
omnichannel-chat-sdk - microsoft/omnichannel-chat-sdk#529
omnichannel-chat-widget - microsoft/omnichannel-chat-widget#830
omnichannel-amsclient - microsoft/omnichannel-amsclient#63

bghcore and others added 3 commits February 11, 2026 17:15
Documents the package architecture, source structure, all 18+ SDK API methods,
3-stage build pipeline (TSC→Babel→Webpack), retry/timeout configuration,
coding patterns, test infrastructure, and integration points.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Remove getContext from timeout test defaultOpt. The getContext flag triggers
BrowserInfo/DeviceInfo/OSInfo calls without spies, which fail because other
test specs corrupt window.navigator via Object.defineProperty without cleanup.
Timeout tests should only verify HTTP timeout error wrapping behavior.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@bghcore bghcore merged commit b20bc7a into microsoft:main Feb 12, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants