Skip to content

@W-21467732 Update siteUrlPathPrefix property to all lowercase#116

Open
jluftglidden-tilt wants to merge 1 commit intoforcedotcom:mainfrom
jluftglidden-tilt:t/experience-sites-platform/w-21467732/update-url-path-prefix-generation
Open

@W-21467732 Update siteUrlPathPrefix property to all lowercase#116
jluftglidden-tilt wants to merge 1 commit intoforcedotcom:mainfrom
jluftglidden-tilt:t/experience-sites-platform/w-21467732/update-url-path-prefix-generation

Conversation

@jluftglidden-tilt
Copy link
Copy Markdown
Contributor

References: Contributing guide · Skill authoring guide · Agent Skills spec

What changed

Why

Notes


Skills

Manual checklist

Description quality

  • Describes what the skill does and the expected output
  • Includes relevant Salesforce domain keywords (Apex, LWC, SOQL, metadata types, etc.)
  • Trigger phrases are specific enough for Vibes to select this skill reliably

Instructions

  • Clear goal statement
  • Step-by-step workflow
  • Validation rules for generated output
  • Defined output / artifact

Context efficiency

  • Core instructions are concise — supporting material lives in templates/, examples/, or docs/ subdirectories
  • No unnecessary background explanation in the body

Automated checks

Enforced by CI (npm run validate:skills) per the Agent Skills spec:

  • Directory is one level deep, named in kebab-case (max 64 chars), contains SKILL.md
  • Frontmatter name matches directory name; description is present, ≥ 20 words, ≤ 1024 characters, and includes trigger language
  • Body is non-empty and under 500 lines
  • Name uses gerund form ⚠ (warning — does not block merge)

|----------|--------|----------------|
| **siteName** | `UpperCamelCase` (e.g., `MyCommunity`) | Ask user or derive from context |
| **siteUrlPathPrefix** | `kebab-case` (e.g., `my-community`) | User-provided, or convert siteName to kebab-case |
| **siteUrlPathPrefix** | `All lowercase` (e.g., `mycommunity`) | User-provided, or convert siteName to all lowercase with alphanumeric characters only |
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Capital letters are allowed in this URL, but this should be fine.

urlName within sfdc_cms__site/*/content.json only allow lowercase letters and numbers though

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We're using siteUrlPathPrefix for the value of urlName in content.json. Hence the all lower case conversion.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants