@salesforce/afv-skills 1.4.0 → 1.5.1

package/skills/generating-apex-test/references/mocking-patterns.md

@@ -0,0 +1,219 @@
+# Mocking Patterns
+
+## HTTP Callout Mocking
+
+Apex doesn't allow real HTTP callouts in tests. Use `HttpCalloutMock` interface.
+
+### Basic Mock Implementation
+
+```apex
+@isTest
+public class MockHttpResponse implements HttpCalloutMock {
+    
+    private Integer statusCode;
+    private String body;
+    
+    public MockHttpResponse(Integer statusCode, String body) {
+        this.statusCode = statusCode;
+        this.body = body;
+    }
+    
+    public HTTPResponse respond(HTTPRequest req) {
+        HttpResponse res = new HttpResponse();
+        res.setStatusCode(statusCode);
+        res.setBody(body);
+        res.setHeader('Content-Type', 'application/json');
+        return res;
+    }
+}
+```
+
+### Using the Mock
+
+```apex
+@isTest
+static void shouldProcessApiResponse_WhenCalloutSucceeds() {
+    // Given
+    String mockResponse = '{"status": "success", "data": [{"id": "123"}]}';
+    Test.setMock(HttpCalloutMock.class, new MockHttpResponse(200, mockResponse));
+    
+    // When
+    Test.startTest();
+    List<ExternalRecord> results = MyIntegrationService.fetchRecords();
+    Test.stopTest();
+    
+    // Then
+    System.assertEquals(1, results.size(), 'Should parse one record from response');
+    System.assertEquals('123', results[0].externalId, 'Should extract correct ID');
+}
+
+@isTest
+static void shouldHandleError_WhenCalloutFails() {
+    // Given
+    String errorResponse = '{"error": "Unauthorized"}';
+    Test.setMock(HttpCalloutMock.class, new MockHttpResponse(401, errorResponse));
+    
+    // When
+    Test.startTest();
+    CalloutResult result = MyIntegrationService.fetchRecords();
+    Test.stopTest();
+    
+    // Then
+    System.assertEquals(false, result.isSuccess, 'Should indicate failure');
+    System.assert(result.errorMessage.contains('Unauthorized'), 'Should capture error');
+}
+```
+
+### Multi-Request Mock
+
+For services making multiple callouts:
+
+```apex
+@isTest
+public class MultiRequestMock implements HttpCalloutMock {
+    
+    private Map<String, HttpResponse> endpointResponses;
+    
+    public MultiRequestMock(Map<String, HttpResponse> responses) {
+        this.endpointResponses = responses;
+    }
+    
+    public HTTPResponse respond(HTTPRequest req) {
+        String endpoint = req.getEndpoint();
+        
+        for (String key : endpointResponses.keySet()) {
+            if (endpoint.contains(key)) {
+                return endpointResponses.get(key);
+            }
+        }
+        
+        // Default 404 if no match
+        HttpResponse res = new HttpResponse();
+        res.setStatusCode(404);
+        res.setBody('{"error": "Not found"}');
+        return res;
+    }
+}
+
+// Usage:
+Map<String, HttpResponse> mocks = new Map<String, HttpResponse>();
+
+HttpResponse authResponse = new HttpResponse();
+authResponse.setStatusCode(200);
+authResponse.setBody('{"token": "abc123"}');
+mocks.put('/oauth/token', authResponse);
+
+HttpResponse dataResponse = new HttpResponse();
+dataResponse.setStatusCode(200);
+dataResponse.setBody('{"records": []}');
+mocks.put('/api/records', dataResponse);
+
+Test.setMock(HttpCalloutMock.class, new MultiRequestMock(mocks));
+```
+
+## StaticResourceCalloutMock
+
+For complex response bodies, store JSON in Static Resources:
+
+```apex
+@isTest
+static void shouldParseComplexResponse() {
+    StaticResourceCalloutMock mock = new StaticResourceCalloutMock();
+    mock.setStaticResource('TestApiResponse'); // Static Resource name
+    mock.setStatusCode(200);
+    mock.setHeader('Content-Type', 'application/json');
+    
+    Test.setMock(HttpCalloutMock.class, mock);
+    
+    Test.startTest();
+    Result r = MyService.callExternalApi();
+    Test.stopTest();
+    
+    System.assertNotEquals(null, r, 'Should parse response');
+}
+```
+
+## Stub API (Enterprise Pattern)
+
+For mocking Apex class dependencies using `System.StubProvider`:
+
+```apex
+@isTest
+public class MyServiceMock implements System.StubProvider {
+    
+    public Object handleMethodCall(
+        Object stubbedObject,
+        String stubbedMethodName,
+        Type returnType,
+        List<Type> paramTypes,
+        List<String> paramNames,
+        List<Object> args
+    ) {
+        if (stubbedMethodName == 'getAccountData') {
+            return new AccountData('Mock Account', 'Active');
+        }
+        return null;
+    }
+}
+
+// Usage in test:
+@isTest
+static void shouldUseAccountData() {
+    MyServiceMock mockProvider = new MyServiceMock();
+    IMyService mockService = (IMyService)Test.createStub(IMyService.class, mockProvider);
+    
+    // Inject mock into class under test
+    MyController controller = new MyController(mockService);
+    
+    Test.startTest();
+    String result = controller.displayAccountInfo();
+    Test.stopTest();
+    
+    System.assert(result.contains('Mock Account'), 'Should use mocked data');
+}
+```
+
+## Email Mocking
+
+Apex sends real emails by default. Use limits to verify:
+
+```apex
+@isTest
+static void shouldSendEmail_WhenTriggered() {
+    Integer emailsBefore = Limits.getEmailInvocations();
+    
+    Test.startTest();
+    MyService.sendNotification(testContact);
+    Test.stopTest();
+    
+    // Verify email was queued (not actually sent in tests)
+    System.assertEquals(
+        emailsBefore + 1, 
+        Limits.getEmailInvocations(), 
+        'One email should be sent'
+    );
+}
+```
+
+## Platform Event Testing
+
+```apex
+@isTest
+static void shouldPublishEvent_WhenRecordCreated() {
+    Test.startTest();
+    
+    // Enable event delivery in test context
+    Test.enableChangeDataCapture();
+    
+    Account acc = TestDataFactory.createAccount(true);
+    
+    // Deliver events
+    Test.getEventBus().deliver();
+    
+    Test.stopTest();
+    
+    // Query platform event trigger results
+    List<EventLog__c> logs = [SELECT Id FROM EventLog__c WHERE AccountId__c = :acc.Id];
+    System.assertEquals(1, logs.size(), 'Event handler should create log record');
+}
+```

package/skills/generating-apex-test/references/test-data-factory.md

@@ -0,0 +1,176 @@
+# TestDataFactory Patterns
+
+## Overview
+
+TestDataFactory is a centralized utility class for creating test records with sensible defaults. It ensures consistent test data across all test classes and reduces duplication.
+
+## Base Template
+
+```apex
+@isTest
+public class TestDataFactory {
+    
+    // ============ ACCOUNTS ============
+    
+    public static List<Account> createAccounts(Integer count, Boolean doInsert) {
+        List<Account> accounts = new List<Account>();
+        for (Integer i = 0; i < count; i++) {
+            accounts.add(new Account(
+                Name = 'Test Account ' + i,
+                BillingStreet = '123 Test St',
+                BillingCity = 'San Francisco',
+                BillingState = 'CA',
+                BillingPostalCode = '94105',
+                BillingCountry = 'USA',
+                Industry = 'Technology',
+                Type = 'Customer'
+            ));
+        }
+        if (doInsert) insert accounts;
+        return accounts;
+    }
+    
+    public static Account createAccount(Boolean doInsert) {
+        return createAccounts(1, doInsert)[0];
+    }
+    
+    // ============ CONTACTS ============
+    
+    public static List<Contact> createContacts(List<Account> accounts, Integer countPerAccount, Boolean doInsert) {
+        List<Contact> contacts = new List<Contact>();
+        Integer index = 0;
+        for (Account acc : accounts) {
+            for (Integer i = 0; i < countPerAccount; i++) {
+                contacts.add(new Contact(
+                    FirstName = 'Test',
+                    LastName = 'Contact ' + index,
+                    Email = 'test.contact' + index + '@example.com',
+                    Phone = '555-000-' + String.valueOf(index).leftPad(4, '0'),
+                    AccountId = acc.Id
+                ));
+                index++;
+            }
+        }
+        if (doInsert) insert contacts;
+        return contacts;
+    }
+    
+    // ============ OPPORTUNITIES ============
+    
+    public static List<Opportunity> createOpportunities(List<Account> accounts, Integer countPerAccount, Boolean doInsert) {
+        List<Opportunity> opps = new List<Opportunity>();
+        Integer index = 0;
+        for (Account acc : accounts) {
+            for (Integer i = 0; i < countPerAccount; i++) {
+                opps.add(new Opportunity(
+                    Name = 'Test Opportunity ' + index,
+                    AccountId = acc.Id,
+                    StageName = 'Prospecting',
+                    CloseDate = Date.today().addDays(30),
+                    Amount = 10000 + (index * 1000)
+                ));
+                index++;
+            }
+        }
+        if (doInsert) insert opps;
+        return opps;
+    }
+    
+    // ============ USERS ============
+    
+    public static User createUser(String profileName, Boolean doInsert) {
+        Profile p = [SELECT Id FROM Profile WHERE Name = :profileName LIMIT 1];
+        String uniqueKey = String.valueOf(DateTime.now().getTime());
+        
+        User u = new User(
+            FirstName = 'Test',
+            LastName = 'User ' + uniqueKey,
+            Email = 'testuser' + uniqueKey + '@example.com',
+            Username = 'testuser' + uniqueKey + '@example.com.test',
+            Alias = 'tuser',
+            TimeZoneSidKey = 'America/Los_Angeles',
+            LocaleSidKey = 'en_US',
+            EmailEncodingKey = 'UTF-8',
+            LanguageLocaleKey = 'en_US',
+            ProfileId = p.Id
+        );
+        if (doInsert) insert u;
+        return u;
+    }
+    
+    // ============ CUSTOM OBJECTS ============
+    
+    // Add methods for your custom objects following the same pattern:
+    // public static List<MyObject__c> createMyObjects(Integer count, Boolean doInsert) { ... }
+}
+```
+
+## Field Override Pattern
+
+Allow callers to override default values:
+
+```apex
+public static Account createAccount(Map<String, Object> fieldOverrides, Boolean doInsert) {
+    Account acc = new Account(
+        Name = 'Test Account',
+        Industry = 'Technology'
+    );
+    
+    // Apply overrides
+    for (String fieldName : fieldOverrides.keySet()) {
+        acc.put(fieldName, fieldOverrides.get(fieldName));
+    }
+    
+    if (doInsert) insert acc;
+    return acc;
+}
+
+// Usage:
+Account acc = TestDataFactory.createAccount(new Map<String, Object>{
+    'Name' => 'Custom Name',
+    'Industry' => 'Healthcare'
+}, true);
+```
+
+## Handling Required Fields and Validation Rules
+
+```apex
+public static Account createAccountWithRequiredFields(Boolean doInsert) {
+    Account acc = new Account(
+        Name = 'Test Account',
+        // Required custom fields
+        External_Id__c = 'EXT-' + String.valueOf(DateTime.now().getTime()),
+        // Fields required by validation rules
+        Phone = '555-123-4567',
+        Website = 'https://example.com'
+    );
+    if (doInsert) insert acc;
+    return acc;
+}
+```
+
+## Record Type Support
+
+```apex
+public static Account createAccountByRecordType(String recordTypeName, Boolean doInsert) {
+    Id recordTypeId = Schema.SObjectType.Account
+        .getRecordTypeInfosByDeveloperName()
+        .get(recordTypeName)
+        .getRecordTypeId();
+    
+    Account acc = new Account(
+        Name = 'Test Account',
+        RecordTypeId = recordTypeId
+    );
+    if (doInsert) insert acc;
+    return acc;
+}
+```
+
+## Best Practices
+
+1. **Always include doInsert parameter** - Allows flexibility for tests that need to modify records before insert
+2. **Use unique identifiers** - Include index or timestamp in Name/Email fields to avoid duplicates
+3. **Set all required fields** - Include all fields required by validation rules
+4. **Return the created records** - Enables chaining and further manipulation
+5. **Create bulk methods first** - Single record methods should call bulk methods with count=1

package/skills/generating-experience-lwr-site/SKILL.md

@@ -7,6 +7,11 @@ description: "Creates, modifies, or manages Salesforce Experience Cloud LWR site
 
 Build and configure Salesforce Experience Cloud Lightning Web Runtime (LWR) sites via metadata (DigitalExperienceConfig, DigitalExperienceBundle, Network, CustomSite, CMS contents).
 
+## IMPORTANT!!
+
+Right after loading this skill, you MUST copy the selected workflows/steps to your plan as a TODO checklist and work on each of the item carefully to ensure correctness.
+You MUST load the relevant reference docs even though they may live outside of user's project folder.
+
 ## Table of Contents
 
 - When to Use
@@ -33,8 +38,8 @@ When working with Experience LWR sites:
 ## Critical Rules
 
 1. Before using any MCP tool, make sure they're actually available. If a tool is missing for the current task, let the user know and pause the current workflow.
-2. **ALWAYS** load the relevant reference docs before doing anything.
-3. **ALWAYS** strictly follow workflows in [Common Workflows](#common-workflows) that match user's requirements. The instructions there should override any conflicting global rules and should have the highest priority over your existing knowledge.
+2. **MUST ALWAYS** load the relevant reference docs before doing anything.
+3. **MUST ALWAYS** strictly follow workflows in [Common Workflows](#common-workflows) that match user's requirements. The instructions there should override any conflicting global rules and should have the highest priority over your existing knowledge.
 4. Flexipage is abstracted away for newer LWR sites with DigitalExperienceBundle, so **NEVER** use any Flexipage-related MCP tool or skills to handle LWR sites' contents.
 
 ## Core Site Properties
@@ -71,7 +76,7 @@ Before doing anything else, note down the following properties from the local pr
 | `sfdc_cms__appPage` | Application page container that groups routes and views | Required; defines the app shell |
 | `sfdc_cms__route` | URL routing definition mapping paths to views | Create one for each page/URL path |
 | `sfdc_cms__view` | Page layout and component structure | Create one for each route; defines page content. Also use to edit existing views (e.g., adding/removing components on a specific page) |
-| `sfdc_cms__brandingSet` | Brand colors, fonts, and styling tokens | Required; defines site-wide styling |
+| `sfdc_cms__brandingSet` | Brand colors, fonts, and styling tokens | Required; defines site-wide styling. Use to create or edit existing branding sets |
 | `sfdc_cms__languageSettings` | Language and localization configuration | Required; defines supported languages |
 | `sfdc_cms__mobilePublisherConfig` | Mobile app publishing settings | Required for mobile app deployment |
 | `sfdc_cms__theme` | Theme definition referencing layouts and branding | Required; one per site |
@@ -79,9 +84,14 @@ Before doing anything else, note down the following properties from the local pr
 
 **Important:** Creating any new pages require BOTH `sfdc_cms__route` AND `sfdc_cms__view`.
 
+#### Object Pages
+
+Object Pages are dedicated pages used to display and manage record-level data for a specific Salesforce entity/object. For example, an custom object "Car" should have "Car_Detail", "Car_List", and "Car_Related_list" views.
+
 ## References
 
 Reference docs within the skill directory. Note that these are **local** and not MCP.
+Before doing anything, you **MUST ALWAYS** load them first if they match user intent.
 
 - [bootstrap-template-byo-lwr.md](docs/bootstrap-template-byo-lwr.md) - Site creation, template defaults
 - [configure-content-route.md](docs/configure-content-route.md) - Route creation (custom/object pages)
@@ -111,32 +121,46 @@ Reference docs within the skill directory. Note that these are **local** and not
 
 **Steps** (Follow the steps sequentially. Do not skip any step before proceeding):
 
-- [ ] Load [configure-content-route.md](docs/configure-content-route.md)
-- [ ] Load [configure-content-view.md](docs/configure-content-view.md)
-- [ ] Load [handle-component-and-region-ids.md](docs/handle-component-and-region-ids.md)
-- [ ] Follow the instructions of the above docs strictly to accomplish user's goal
+- [ ] MUST read [configure-content-route.md](docs/configure-content-route.md)
+- [ ] MUST read [configure-content-view.md](docs/configure-content-view.md)
+- [ ] MUST read [handle-component-and-region-ids.md](docs/handle-component-and-region-ids.md)
 
 ### Adding UI Components to Pages
 
 **Steps** (Follow the steps sequentially. Do not skip any step before proceeding):
 
-- [ ] Read and follow [handle-ui-components.md](docs/handle-ui-components.md) to add LWCs to LWR sites.
-- [ ] Load and follow [handle-component-and-region-ids.md](docs/handle-component-and-region-ids.md) to handle id generation
-- [ ] Read and follow [configure-content-themeLayout.md](docs/configure-content-themeLayout.md) if a component has one of the following requirements:
+- [ ] MUST read [handle-ui-components.md](docs/handle-ui-components.md) to add LWCs to LWR sites.
+- [ ] MUST read [handle-component-and-region-ids.md](docs/handle-component-and-region-ids.md) to handle id generation
+- [ ] MUST read [configure-content-themeLayout.md](docs/configure-content-themeLayout.md) if a component has one of the following requirements:
   - needs to be "sticky" and persistent across pages
   - is used as a theme layout
 
+### Creating Page Layouts / Container Components
+
+**Steps** (Follow the steps sequentially. Do not skip any step before proceeding):
+
+- [ ] MUST read [handle-ui-components.md](docs/handle-ui-components.md)
+
 ### Creating Theme Layouts
 
 **Steps** (Follow the steps sequentially. Do not skip any step before proceeding):
 
-- [ ] Read and follow strictly [configure-content-themeLayout.md](docs/configure-content-themeLayout.md).
+- [ ] Check with user whether this new theme layout reuses an existing theme layout component or requires a new one.
+- [ ] MUST read [handle-ui-components.md](docs/handle-ui-components.md) if creating a new theme layout component.
+- [ ] MUST read [configure-content-themeLayout.md](docs/configure-content-themeLayout.md).
+- [ ] MUST read [configure-content-view.md](docs/configure-content-view.md) if need to apply theme layout to pages
+
+### Applying/Setting Theme Layouts
+
+**Steps** (Follow the steps sequentially. Do not skip any step before proceeding):
+
+- [ ] MUST read [configure-content-view.md](docs/configure-content-view.md)
 
 ### Configuring Branding
 
 **Steps** (Follow the steps sequentially. Do not skip any step before proceeding):
 
-- [ ] Read and follow strictly [configure-content-brandingSet.md](docs/configure-content-brandingSet.md) to configure background colors, foreground colors, button colors, and other branding colors that affect all pages.
+- [ ] MUST read [configure-content-brandingSet.md](docs/configure-content-brandingSet.md) to configure background colors, foreground colors, button colors, and other branding colors that affect all pages.
 
 ### CUD Operations on DigitalExperience Contents
 
@@ -145,8 +169,8 @@ Reference docs within the skill directory. Note that these are **local** and not
 **Steps** (Follow the steps sequentially. Do not skip any step before proceeding):
 
 - [ ] Determine what content types the user wants to modify
-- [ ] Read and follow strictly the reference doc related to the target content types if the doc exists. e.g., if modifying `sfdc_cms__route`, load [configure-content-route.md](docs/configure-content-route.md).
-- [ ] **Always** Read [handle-component-and-region-ids.md](docs/handle-component-and-region-ids.md) if creating or modifying view or theme layout
+- [ ] MUST read the reference doc related to the target content types if the doc exists. e.g., if modifying `sfdc_cms__route`, load [configure-content-route.md](docs/configure-content-route.md).
+- [ ] MUST read [handle-component-and-region-ids.md](docs/handle-component-and-region-ids.md) if creating or modifying view or theme layout
 - [ ] **Always** Call `execute_metadata_action` to get the schema and examples for that content type **after** loading the corresponding reference docs.
   - **Call once per content type per user request**: If you're creating/modifying multiple items of the same content type (e.g., creating 3 routes), you only need to call `execute_metadata_action` ONCE for that content type. Reuse the schema and examples for all items of that type within the same user request.
   - For each unique content type you need to work with, **always** call `execute_metadata_action` using the following:
@@ -162,9 +186,11 @@ Reference docs within the skill directory. Note that these are **local** and not
 }
 ```
 
-### Retrieving Site URLs After Deployment
+### Retrieving Site Preview and Builder URLs After Deployment
+
+**Use when** user requests to preview a site, access a builder site, or after successfully deploying a site.
 
-After successfully deploying the site using `sf project deploy`, use the `execute_metadata_action` MCP tool to get the preview and builder URLs:
+Use the `execute_metadata_action` MCP tool to get the preview and builder URLs:
 
 ```json
 {

package/skills/generating-experience-lwr-site/docs/configure-content-brandingSet.md

@@ -6,6 +6,7 @@
 
 - Core Principles
 - Generation Guidelines
+- Editing Existing Branding Sets
 - Branding Property Patterns
 
 ## Core Principles
@@ -58,7 +59,11 @@ The `content.json` file must contain:
 - `title`: **Required**. Human-readable display title (e.g., Branding Set).
   - Maximum length is **100 characters**.
   - Must be **unique** within the space's brandingSet content items.
-- `contentBody`: Include all `required` properties from `schemaDefinition`. Use `examplesOfContentType` for reference.
+- `contentBody`: Include all `required` properties from `schemaDefinition`. 
+  1. **Seed**: Always call `execute_metadata_action` with `shouldIncludeExamples: true`. Copy the *entire* example object from `examplesOfContentType[0]` into `content.json`. **NEVER** start from a minimal stub.
+  2. **Recalculate (CRITICAL STOP)**: You MUST stop and perform explicit changes for dependent tokens BEFORE generating JSON.
+    - [] Refer to "Branding Property Patterns" for detailed calculations.
+
   - `brandingSetType`: Represents whether the color palette is for the entire site or a specific section.
     - `APP`: The branding set applies to the entire site. There can be only one branding set of this type.
     - `SCOPED`: A `SCOPED` branding set can be applied only to a section component for granular overrides.
@@ -70,10 +75,6 @@ The `content.json` file must contain:
     - **Patterns**: See the "Branding Property Patterns" section for details on value relationships.
 - `urlName`: Lowercase with hyphens (e.g., `branding-set`)
 
-**Rules**:
-
-- Before any actions, *always* call `execute_metadata_action` to get the full schema and examples per the skill document.
-
 ### 4. Naming Conventions Summary
 
 | Field | Format | Example |
@@ -84,9 +85,18 @@ The `content.json` file must contain:
 
 ### 5. Generation Checklist
 
-- [ ] Directory and `_meta.json` follow naming conventions (1, 2)
-- [ ] `content.json` has all required fields (3)
+- [ ] Directory and `_meta.json` follow naming conventions
+- [ ] `content.json` has all required fields
 - [ ] `contentBody` follows the schema provided by `execute_metadata_action`
+- [ ] **STOP AND VERIFY**: `contentBody.values` honors all **Branding Property Patterns** defined below and explicitly recalculated and updated all dependent tokens based on any token updates requested by the user.
+
+## Editing Existing Branding Sets
+
+Use this section when modifying existing branding sets under the `sfdc_cms__brandingSet` directory.
+
+### Editing Checklist
+
+- [ ] Ensure all modified branding properties honor the **Branding Property Patterns** defined below.
 
 ## Branding Property Patterns
 

package/skills/generating-experience-lwr-site/docs/configure-content-themeLayout.md

@@ -72,6 +72,7 @@ The `content.json` file must contain:
 - `contentBody`: Include all `required` properties from `schemaDefinition`. Use `examplesOfContentType` for reference.
 - Do not add additional fields.
 - `urlName`: URL identifier (lowercase, words separated by dashes e.g., "scoped-header-and-footer")
+- `contentBody.compnent.definition`: The actual theme layout component that displays/renders the layout and includes theme region components.
 
 **Rules**:
 
@@ -130,7 +131,7 @@ When generating a new theme layout, ensure:
 - [ ] `urlName` uses lowercase with hyphens (V)
 - [ ] `title` is human-readable (V)
 - [ ] `sfdc_cms__theme/[THEME_API_NAME]/content.json` updated by appending a new `contentBody.layouts` mapping (VI)
-- [ ] **CRITICAL**: Complete all the UUID generation steps. See `docs/handle-component-and-region-ids.md`
+- [ ] **CRITICAL**: Complete all the UUID generation steps. See [handle-component-and-region-ids.md](docs/handle-component-and-region-ids.md)
 
 ## Purpose B: Editing Existing Theme Layouts
 

package/skills/generating-experience-lwr-site/docs/configure-content-view.md

@@ -49,7 +49,7 @@ The `_meta.json` file must contain:
 
 ### Theme Layout Type (All Views)
 
-The `contentBody.themeLayoutType` field specifies which theme layout to use for the view.
+The `contentBody.themeLayoutType` field specifies which theme layout to use for the view. There can only be one per view.
 
 - **Default**: `"Inner"` - Use this default if the user does not specify a layout OR if the lookup fails to find a matching layoutType
 - **Lookup**: To find valid values:
@@ -126,7 +126,7 @@ The route's `activeViewId` must match the view's directory name exactly.
 - [ ] Directory and `_meta.json` follow structure (see Directory Structure, _meta.json Structure)
 - [ ] `content.json` has all required fields (A.1)
 - [ ] Component structure correct with both regions (A.1)
-- [ ] **CRITICAL**: Complete all the UUID generation steps. see `docs/handle-component-and-region-ids.md`
+- [ ] **CRITICAL**: Complete all the UUID generation steps. see [handle-component-and-region-ids.md](docs/handle-component-and-region-ids.md)
 - [ ] `viewType` matches route's `routeType` (CRITICAL)
 
 ### PART B: OBJECT PAGES
@@ -218,7 +218,7 @@ The route's `activeViewId` must match the view's directory name exactly. The `vi
 - [ ] `viewType` matches route's `routeType` for all three views (CRITICAL)
 - [ ] Component structure correct with both regions (see A.1)
 - [ ] SEO assistant configured correctly per view type (B.4)
-- [ ] **CRITICAL**: Complete both UUID generation steps. see `docs/handle-component-and-region-ids.md`
+- [ ] **CRITICAL**: Complete both UUID generation steps. see [handle-component-and-region-ids.md](docs/handle-component-and-region-ids.md)
 
 ## Purpose B: Editing Existing Views
 

package/skills/generating-experience-react-site/SKILL.md

@@ -51,6 +51,17 @@ Use the default templates in the docs below. Values in `{braces}` are resolved p
 | DigitalExperienceBundle | [configure-metadata-digital-experience-bundle.md](docs/configure-metadata-digital-experience-bundle.md) |
 | DigitalExperience (sfdc_cms__site) | [configure-metadata-digital-experience.md](docs/configure-metadata-digital-experience.md) |
 
+### Execution Note for Step 3: Load and use the docs
+- Agents MUST read the full contents of each docs/*.md file referenced in Step 3 before attempting to populate metadata fields.
+- Use your platform's file-read tool (for example, `read_file`) to load these files in full, then perform placeholder substitution for values in `{braces}` using the resolved properties from Step 1.
+- Files to load:
+  - `docs/configure-metadata-network.md`
+  - `docs/configure-metadata-custom-site.md`
+  - `docs/configure-metadata-digital-experience-config.md`
+  - `docs/configure-metadata-digital-experience-bundle.md`
+  - `docs/configure-metadata-digital-experience.md`
+- Read entire file contents, replace placeholders (e.g. `{siteName}`) with the resolved values, then use the expanded templates to populate the metadata XML/JSON content.
+  
 ### Step 4: Resolve Additional Configurations
 Address any extra configurations the user requests. Use the schemas returned by `get_metadata_api_context` in Step 2 to understand each field's purpose, and update only the minimum necessary fields.