chore(branding): remove PPDS.Tools / PPDS ALM, rename SDK to libraries - #19
Conversation
Brings .claude/rules/branding.md in sync with what actually ships at v1. The freshly-pruned CLAUDE.md (#18) points to this file as the canonical name list, so stale content here actively misleads any AI session authoring docs. Changes: - Remove PPDS.Tools and PPDS ALM from the Components table (not shipping at v1, can be re-added when they do) - Remove ppds-tools and ppds-alm from the Repositories table - Remove the PSGallery link (no PPDS.Tools package) - Main repo description: SDK -> Libraries - Value prop: "standard SDK patterns" -> "standard Dataverse SDK patterns" (disambiguates Microsoft's SDK from PPDS libraries) - New Terminology section explicitly calling out libraries-vs-SDK and the NoSdkInPresentationAnalyzer rule that enforces the same convention in code Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Code Review
This pull request updates the branding and terminology guidelines in .claude/rules/branding.md. Key changes include removing references to PowerShell and ALM components, updating repository tables, and introducing a 'Terminology' section to distinguish between PPDS 'libraries' and the Microsoft Dataverse 'SDK'. Review feedback suggests reordering the component list in the repository table for consistency and ensuring the 'PPDS' abbreviation is consistently capitalized in the documentation.
| | Repo | GitHub Name | Display Name | | ||
| |------|-------------|--------------| | ||
| | Main | power-platform-developer-suite | PPDS (SDK + CLI + TUI + Extension + MCP) | | ||
| | Main | power-platform-developer-suite | PPDS (Libraries + CLI + TUI + Extension + MCP) | |
There was a problem hiding this comment.
The list of components in the repository description should follow the order established in the 'Components' table (lines 19-22) for better consistency. Additionally, 'TUI' should be grouped as a feature of the 'CLI' component. Please ensure these rules are centralized in the primary AI instructions file to avoid duplication and maintain a single source of truth.
| | Main | power-platform-developer-suite | PPDS (Libraries + CLI + TUI + Extension + MCP) | | |
| | Main | power-platform-developer-suite | PPDS (CLI + TUI + Extension + Libraries + MCP) | |
References
- Centralize global rules in a primary file to maintain a single source of truth and conserve context.
|
|
||
| ## Terminology | ||
|
|
||
| - Use **"libraries"** (not "SDK") to refer to PPDS NuGet packages (`PPDS.Auth`, `PPDS.Dataverse`, `PPDS.Migration`, `PPDS.Plugins`). "SDK" is reserved for Microsoft's Dataverse SDK (`ServiceClient`, `IOrganizationService`) that the PPDS libraries sit on top of. Disambiguation matters — sentences like "the SDK handles throttling" are ambiguous in this codebase. The `NoSdkInPresentationAnalyzer` rule in the ppds monorepo enforces the same convention in code. |
There was a problem hiding this comment.
Capitalize the product abbreviation as 'PPDS' for consistency with branding guidelines. Additionally, use 'libraries' for PPDS packages and 'SDK' for the underlying Microsoft Dataverse SDK to avoid ambiguity. These rules should be centralized in the primary AI instructions file to avoid duplication.
| - Use **"libraries"** (not "SDK") to refer to PPDS NuGet packages (`PPDS.Auth`, `PPDS.Dataverse`, `PPDS.Migration`, `PPDS.Plugins`). "SDK" is reserved for Microsoft's Dataverse SDK (`ServiceClient`, `IOrganizationService`) that the PPDS libraries sit on top of. Disambiguation matters — sentences like "the SDK handles throttling" are ambiguous in this codebase. The `NoSdkInPresentationAnalyzer` rule in the ppds monorepo enforces the same convention in code. | |
| - Use "libraries" (not "SDK") to refer to PPDS NuGet packages (PPDS.Auth, PPDS.Dataverse, PPDS.Migration, PPDS.Plugins). "SDK" is reserved for Microsoft's Dataverse SDK (ServiceClient, IOrganizationService) that the PPDS libraries sit on top of. Disambiguation matters — sentences like "the SDK handles throttling" are ambiguous in this codebase. The NoSdkInPresentationAnalyzer rule in the PPDS monorepo enforces the same convention in code. |
References
- Centralize global rules in a primary file to maintain a single source of truth and conserve context.
Summary
Brings
.claude/rules/branding.mdin sync with what actually ships at v1. CLAUDE.md #18 recently pruned to point at this file as the canonical name list, so staleness here now actively misleads any AI session authoring docs.Changes
PowerShell | PPDS.ToolsandALM | PPDS ALMrows (not shipping at v1; add back when they do)ppds-toolsandppds-almrows; update Main repo description fromPPDS (SDK + CLI + TUI + Extension + MCP)toPPDS (Libraries + CLI + TUI + Extension + MCP)librariesvsSDK, noting theNoSdkInPresentationAnalyzerrule in the ppds monorepo that enforces the same convention in codestandard SDK patterns→standard Dataverse SDK patterns(disambiguates Microsoft's SDK from PPDS libraries)Test plan
.claude/rules/is ignored), so no build check required🤖 Generated with Claude Code