Skip to content

chore(branding): remove PPDS.Tools / PPDS ALM, rename SDK to libraries - #19

Merged
joshsmithxrm merged 1 commit into
mainfrom
chore/branding-cleanup
Apr 19, 2026
Merged

chore(branding): remove PPDS.Tools / PPDS ALM, rename SDK to libraries#19
joshsmithxrm merged 1 commit into
mainfrom
chore/branding-cleanup

Conversation

@joshsmithxrm

Copy link
Copy Markdown
Owner

Summary

Brings .claude/rules/branding.md in 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

  • Components table — remove PowerShell | PPDS.Tools and ALM | PPDS ALM rows (not shipping at v1; add back when they do)
  • Repositories table — remove ppds-tools and ppds-alm rows; update Main repo description from PPDS (SDK + CLI + TUI + Extension + MCP) to PPDS (Libraries + CLI + TUI + Extension + MCP)
  • Links — remove PSGallery link (no PPDS.Tools package to point at)
  • New Terminology section — explicit rule on libraries vs SDK, noting the NoSdkInPresentationAnalyzer rule in the ppds monorepo that enforces the same convention in code
  • Value propsstandard SDK patternsstandard Dataverse SDK patterns (disambiguates Microsoft's SDK from PPDS libraries)

Test plan

  • File is not rendered by Docusaurus (.claude/rules/ is ignored), so no build check required
  • Verify CLAUDE.md link target still resolves

🤖 Generated with Claude Code

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>
@joshsmithxrm
joshsmithxrm merged commit 8ff3552 into main Apr 19, 2026
1 check passed
@joshsmithxrm
joshsmithxrm deleted the chore/branding-cleanup branch April 19, 2026 12:21
@github-project-automation github-project-automation Bot moved this from Todo to Done in PPDS Roadmap Apr 19, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Comment thread .claude/rules/branding.md
| 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) |

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

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.

Suggested change
| Main | power-platform-developer-suite | PPDS (Libraries + CLI + TUI + Extension + MCP) |
| Main | power-platform-developer-suite | PPDS (CLI + TUI + Extension + Libraries + MCP) |
References
  1. Centralize global rules in a primary file to maintain a single source of truth and conserve context.

Comment thread .claude/rules/branding.md

## 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.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

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.

Suggested change
- 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
  1. Centralize global rules in a primary file to maintain a single source of truth and conserve context.

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

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant