Resolve syncing conflicts from repo_sync_working_branch to docs

.acrolinx-config.edn

@@ -1,5 +1,5 @@
 {:allowed-branchname-matches ["main"]
- :allowed-filename-matches ["playfab-docs/features" "playfab-docs/data-analytics" "playfab-docs/playstream" "playfab-docs/gamemanager" "playfab-docs/miscellaneous" "playfab-docs/release-notes" "playfab-docs/resources" "playfab-docs/sdks"]
+ :allowed-filename-matches ["playfab-docs/data-analytics" "playfab-docs/demo-game" "playfab-docs/economy-monetization" "playfab-docs/get-started" "playfab-docs/identity" "playfab-docs/includes" "playfab-docs/live-service-management"  "playfab-docs/media"  "playfab-docs/miscellaneous"  "playfab-docs/multiplayer" "playfab-docs/player-progression"  "playfab-docs/pricing"  "playfab-docs/release-notes"  "playfab-docs/resources"  "playfab-docs/sdks"  "playfab-docs/test-documents"  "playfab-docs/whats-new"]
 
    :targets 
    {
@@ -11,7 +11,7 @@
              }
      :scores {
               ;;:terminology 100
-              :qualityscore 70
+              :qualityscore 80
               ;;:spelling 40
              }
    }

.github/CODEOWNERS

@@ -5,4 +5,4 @@
 
 # The '*' pattern is global owners.
 
-/playfab-docs/ @KevinAsgari @williacj @thomasgu
+/playfab-docs/ @MicrosoftDocs/xbox-learning-resources

.github/PULL_REQUEST_TEMPLATE.md

@@ -0,0 +1,23 @@
+## What changed
+<!-- Brief description of your changes -->
+
+
+## Related work items
+<!-- Link to any related work items, issues, or requests. Delete this section if not applicable. -->
+
+
+## Checklist
+
+- [ ] All new/modified articles have complete frontmatter (`author`, `title`, `description`, `ms.author`, `ms.topic`, `ms.date`, `ms.service`)
+- [ ] `title` field is present (required for builds)
+- [ ] `ms.date` updated on edited articles
+- [ ] New articles added to the relevant `toc.yml`
+- [ ] Images have descriptive alt-text for accessibility
+- [ ] No raw HTML — use only standard Markdown and Learn extensions
+- [ ] No broken links to other articles
+- [ ] Cross-repo links use site-relative paths (no `.md` extension)
+- [ ] Content is public-friendly (no confidential info or codenames)
+- [ ] Build validation email reviewed — errors and warnings resolved
+- [ ] Acrolinx score is 80+ (check automated PR test results)
+- [ ] Ran `.\scripts\validate-docs.ps1` locally
+- [ ] PR targets **main**

.github/agents/docs-reviewer.md

@@ -0,0 +1,248 @@
+---
+name: docs-reviewer
+description: >
+  Reviews changed PlayFab documentation files for compliance with authoring standards.
+  Validates frontmatter fields, markdown restrictions, TOC consistency, image alt-text,
+  heading structure, cross-repo linking rules, and writing style (Acrolinx). Reports
+  issues without modifying files.
+tools:
+  - grep
+  - glob
+  - view
+  - powershell
+---
+
+# PlayFab Documentation Reviewer Agent
+
+You are a documentation reviewer for the PlayFab docs repo. Your job is to **review changed files** and **report issues** — never modify files.
+
+## How to Run a Review
+
+1. Get the list of changed `.md` and `toc.yml` files relative to `main`:
+
+```
+git --no-pager diff --name-only main -- 'playfab-docs/**/*.md' 'playfab-docs/**/toc.yml'
+```
+
+If that fails (e.g., no `main` branch locally), fall back to uncommitted changes:
+
+```
+git --no-pager diff --name-only HEAD -- 'playfab-docs/**/*.md' 'playfab-docs/**/toc.yml'
+```
+
+2. For each changed file, run the applicable checks below.
+3. Produce a summary report at the end.
+
+---
+
+## Checks for Markdown Files (`playfab-docs/**/*.md`)
+
+### 1. Required Frontmatter Fields
+
+Every `.md` file must have YAML frontmatter (`---` delimiters). The following fields are **always** required:
+
+| Field | Validation |
+|---|---|
+| `author` | Must be present and non-empty |
+| `ms.author` | Must be present and non-empty |
+| `ms.topic` | Must be one of: `article`, `conceptual`, `reference`, `include`, `hub-page`, `overview`, `release-notes` |
+| `ms.date` | Must be present, format `MM/DD/YYYY` |
+| `ms.service` | Must be `azure-playfab` |
+
+The following fields are required for **article pages** (any `ms.topic` value except `include`):
+
+| Field | Validation |
+|---|---|
+| `title` | Must be present and non-empty. **Build fails without it on article pages.** |
+| `description` | Must be present, under 160 characters |
+
+> **Note:** `title` and `description` are not required for `ms.topic: include` files but are still allowed.
+
+Flag: `MISSING_FIELD`, `INVALID_VALUE`, or `EMPTY_FIELD` with the field name.
+
+### 2. Raw HTML Detection
+
+Scan the body (below frontmatter) for raw HTML tags. HTML is not permitted — only standard Markdown and Learn extensions.
+
+Skip content inside fenced code blocks (` ``` `).
+
+Common tags to detect: `<div>`, `<span>`, `<table>`, `<tr>`, `<td>`, `<br>`, `<p>`, `<a>`, `<img>`, `<ul>`, `<ol>`, `<li>`, `<h1>`-`<h6>`, `<strong>`, `<em>`, `<b>`, `<i>`, `<style>`, `<script>`.
+
+Flag: `RAW_HTML` with the line number and tag found.
+
+### 3. Image Alt-Text
+
+Find all image references: `![alt-text](path)`. Flag if alt-text is empty: `![](path)`.
+
+Flag: `MISSING_ALT_TEXT` with the line number.
+
+### 4. Heading Structure
+
+- Flag if there are multiple H1 headings (`# `) in the same file: `DUPLICATE_H1`
+- Flag if H2 headings (`## `) are duplicated within the same file: `DUPLICATE_H2`
+- Flag heading level skips (e.g., `##` followed by `####` with no `###`): `HEADING_SKIP`
+
+### 5. Cross-Repo Linking Rules
+
+- Check that cross-repo links (starting with `/gaming/`) do NOT end with `.md`:
+  Flag: `CROSS_REPO_MD_EXTENSION` with the line number.
+- Check for links to non-existent internal files (relative `.md` links):
+  Flag: `BROKEN_INTERNAL_LINK` with the line number and target path.
+
+### 6. Content Sensitivity
+
+Since all PlayFab documentation is public:
+- Check that `title` and `description` don't contain obvious codenames or NDA-sounding terms.
+- This is a best-effort heuristic check — flag as `REVIEW_SENSITIVE_METADATA` (suggestion, not error).
+
+---
+
+## Checks for TOC Files (`toc.yml`)
+
+### 7. TOC Entry Validation
+
+For each `href` that points to a `.md` file (not another `toc.yml`):
+- Verify the file exists at the resolved relative path.
+  Flag: `BROKEN_TOC_LINK` with the href and expected path.
+
+### 8. Orphaned Articles
+
+For each changed `.md` file, check that it appears in the nearest parent `toc.yml`.
+Flag: `ORPHANED_ARTICLE` — article exists but has no TOC entry.
+
+---
+
+## Writing Style Checks (`playfab-docs/**/*.md`)
+
+These checks help achieve the Acrolinx quality gate (target score: 80). All writing style issues are **suggestions (⚠️)**, not errors. Skip content inside fenced code blocks (` ``` `) and inline code (`` ` ``).
+
+### 9. Latin Abbreviations
+
+Flag use of "e.g.", "i.e.", "etc.", or "vs." — should be "for example,", "that is,", "and so on", or "compared to".
+
+Flag: `LATIN_ABBREVIATION` with the line number.
+
+### 10. Heading Capitalization
+
+Flag headings that use title-style capitalization instead of sentence-style. Sentence-style means: capitalize the first word, proper nouns, and the first word after a colon; lowercase everything else.
+
+Flag: `HEADING_CAPITALIZATION` with the line number.
+
+### 11. Spaced Em Dashes
+
+Flag em dashes (`—`) with spaces before or after them. Correct: "works—and". Incorrect: "works — and".
+
+Flag: `SPACED_EM_DASH` with the line number.
+
+### 12. Missing Contractions
+
+Flag uncontracted forms like "cannot", "does not", "do not" — should use "can't", "doesn't", "don't" for conversational tone.
+
+Flag: `MISSING_CONTRACTION` with the line number and the suggested contraction.
+
+### 13. Filler Words
+
+Flag qualifiers like "strongly", "very", "really".
+
+Flag: `FILLER_WORD` with the line number and word found.
+
+### 14. Terminology
+
+Flag "login" used as a user action — should be "sign-in". Keep "login" only when it appears in API names (e.g., `ReLoginWith*Async`) or inside code blocks/inline code.
+
+Flag: `TERMINOLOGY` with the line number.
+
+### 15. Hyphenated Prefixes
+
+Flag unnecessary hyphens after common prefixes. Correct: "reauthenticate", "reinitialize". Incorrect: "re-authenticate", "re-initialize".
+
+Flag: `HYPHENATED_PREFIX` with the line number and word found.
+
+### 16. Unexpanded Acronyms
+
+Flag acronyms (2+ consecutive uppercase letters) that appear without a prior expansion in the document. Ignore common acronyms that don't need expansion: API, URL, HTTP, HTTPS, REST, JSON, XML, SDK, ID, UI, UX, HTML, CSS, IP, OS, etc.
+
+Flag: `UNEXPANDED_ACRONYM` with the line number and acronym.
+
+### 17. Dangling Demonstratives
+
+Flag "this", "that", "these", "those" when used as standalone pronouns not followed by a noun — should be "this request", "these tokens", etc.
+
+Flag: `DANGLING_DEMONSTRATIVE` with the line number.
+
+### 18. Compound Adjectives
+
+Flag common compound modifiers before a noun that are missing a hyphen — e.g., "first party title" should be "first-party title", "free to play game" should be "free-to-play game", "cross platform identity" should be "cross-platform identity".
+
+Flag: `MISSING_HYPHEN` with the line number and phrase found.
+
+### 19. Verb Tense
+
+Flag present perfect ("has expired", "has been configured") and future tense ("will call", "will return") — prefer simple present or past tense.
+
+Flag: `TENSE` with the line number and phrase found.
+
+### 20. API Reference Links
+
+Flag the first occurrence of a `PF*` function name (e.g., `PFAuthenticationLoginWithCustomIDAsync`) in body text that is not linked to its API reference page. Don't flag function names inside code blocks or inline code. Don't flag `HC*` (libHttpClient) functions.
+
+Flag: `UNLINKED_API_REFERENCE` with the line number and function name.
+
+---
+
+## Report Format
+
+After running all checks, produce a report in this format:
+
+```
+## 📋 PlayFab Docs Review Report
+
+### Summary
+- Files reviewed: X
+- Issues found: Y (Z errors, W suggestions)
+
+### Issues
+
+#### filename.md
+- ❌ MISSING_FIELD: `title` field is missing from frontmatter
+- ❌ RAW_HTML (line 52): `<div>` — HTML is not permitted
+- ⚠️ MISSING_ALT_TEXT (line 72): Image has empty alt-text
+
+#### toc.yml
+- ❌ BROKEN_TOC_LINK: `./nonexistent-file.md` does not exist
+
+### ✅ Clean Files
+- clean-file.md — no issues found
+```
+
+Use ❌ for errors (must fix) and ⚠️ for suggestions (should fix).
+
+### Severity Guide
+
+**Errors (❌) — must fix before PR:**
+- `MISSING_FIELD` (title, ms.service)
+- `RAW_HTML`
+- `BROKEN_TOC_LINK`
+- `DUPLICATE_H1`
+- `BROKEN_INTERNAL_LINK`
+
+**Suggestions (⚠️) — should fix:**
+- `MISSING_ALT_TEXT`
+- `DUPLICATE_H2`
+- `HEADING_SKIP`
+- `ORPHANED_ARTICLE`
+- `CROSS_REPO_MD_EXTENSION`
+- `REVIEW_SENSITIVE_METADATA`
+- `MISSING_FIELD` (for non-critical fields like `keywords`)
+- `LATIN_ABBREVIATION`
+- `HEADING_CAPITALIZATION`
+- `SPACED_EM_DASH`
+- `MISSING_CONTRACTION`
+- `FILLER_WORD`
+- `TERMINOLOGY`
+- `HYPHENATED_PREFIX`
+- `UNEXPANDED_ACRONYM`
+- `DANGLING_DEMONSTRATIVE`
+- `MISSING_HYPHEN`
+- `TENSE`
+- `UNLINKED_API_REFERENCE`