Skip to content

Add automated diff review CI signal#1092

Open
yutingye wants to merge 3 commits intomainfrom
export-D95303739
Open

Add automated diff review CI signal#1092
yutingye wants to merge 3 commits intomainfrom
export-D95303739

Conversation

@yutingye
Copy link
Contributor

@yutingye yutingye commented Mar 5, 2026

Differential Revision: D95303739

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Mar 5, 2026
@meta-codesync
Copy link
Contributor

meta-codesync bot commented Mar 5, 2026

@yutingye has exported this pull request. If you are a Meta employee, you can view the originating Diff in D95303739.

meta-codesync bot pushed a commit that referenced this pull request Mar 5, 2026
Summary: Pull Request resolved: #1092

Differential Revision: D95303739
@meta-codesync meta-codesync bot force-pushed the export-D95303739 branch from 9bbc8a1 to 535b2eb Compare March 5, 2026 07:30
meta-codesync bot pushed a commit that referenced this pull request Mar 5, 2026
Summary: Pull Request resolved: #1092

Differential Revision: D95303739
@meta-codesync meta-codesync bot force-pushed the export-D95303739 branch from 535b2eb to fabbef7 Compare March 5, 2026 09:02
meta-codesync bot pushed a commit that referenced this pull request Mar 5, 2026
Summary: Pull Request resolved: #1092

Differential Revision: D95303739
@meta-codesync meta-codesync bot force-pushed the export-D95303739 branch from fabbef7 to 6f41c50 Compare March 5, 2026 09:41
yutingye added 3 commits March 5, 2026 01:44
Summary:
Add Claude Code support and establish a self-maintaining AI config system
for the momentum library. The system uses a shared source of truth pattern
where each piece of project knowledge has exactly one owning file, and all
other files reference it.

New files:
- CLAUDE.md and a public CLAUDE.md for internal and OSS Claude Code support
- Slash commands for build, test, and pre-diff verification
- Config sync conventions with ownership map and anti-patterns
- Open source export guidelines covering diff summaries and export process

Modified files:
- Updated project structure tree, added Windows build mode, added reviewer rule
- Deduplicated build commands, added downstream dependency testing rule
- Expanded pre-diff checklist with references to all rules

The config system is self-enforcing: the maintenance rule fires whenever
any AI config file is edited, and it covers its own update triggers.

Differential Revision: D95275618
Summary:
Add Claude Code support for pymomentum, following the same shared source
of truth pattern established in the momentum library.

New files:
- CLAUDE.md for Claude Code project instructions
- .claude/commands/ for build, test, and verification slash commands
- .llms/rules/ai_config_maintenance.md for config sync conventions

Modified files:
- .llms/rules/overview.md: updated structure tree, added reviewer rule
- .llms/rules/code_style.md: deduplicated build commands, added downstream dep testing
- .llms/commands/check.md: expanded with references to all rules files

OSS export rules and C++ style are shared with momentum and referenced
rather than duplicated.

Differential Revision: D95273003
Summary: Pull Request resolved: #1092

Differential Revision: D95303739
meta-codesync bot pushed a commit that referenced this pull request Mar 5, 2026
Summary: Pull Request resolved: #1092

Differential Revision: D95303739
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot. fb-exported meta-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant