Commit 8949e3a
Add comprehensive historical alias verification and fix 8 missing aliases (#16295)
* Add comprehensive historical alias verification scripts
This adds a new set of scripts to verify ALL historical paths in git history
have proper aliases, catching missing aliases that the branch-diff verification
misses (pre-reorg moves, multi-hop moves, cross-branch moves).
New scripts:
- verify-all-historical-aliases.py: Scans complete git history (6 months)
for all file moves, checks both frontmatter aliases and S3 redirects
- generate-historical-fixes.py: Parses verification output and generates
fix data in JSON format
- apply-historical-fixes.py: Applies fixes by updating frontmatter aliases
Updated README.md with comprehensive documentation of the new workflow,
including when to use historical verification vs branch verification.
Fixes missing aliases from pre-reorg moves that happened on master before
the documentation reorganization branch was created.
Related: #16292, #16119
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
* Add git similarity detection to catch low-similarity renames
The original script missed files that were moved and significantly rewritten
in the same commit (recorded as DELETE+ADD instead of RENAME). This updates
the script to use git's -M30% flag, which detects renames even when files
are only 30% similar.
This now catches the missing OIDC files from the May 2024 documentation
revamp where content was significantly changed during the move:
- /docs/pulumi-cloud/access-management/oidc/provider/aws/
- /docs/pulumi-cloud/access-management/oidc/provider/azure/
- /docs/pulumi-cloud/access-management/oidc/provider/gcp/
- /docs/pulumi-cloud/access-management/oidc/provider/ (index)
Results:
- Before: 82 files with 82 missing aliases
- After: 92 files with 93 missing aliases (+10 files, +11 aliases)
Updated README to document the 30% similarity detection feature and
why it's necessary for catching content revamps.
Fixes #16292 (partially - provides tooling to detect the issue)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
* Fix false positives from development branch paths
The script was checking ALL branches (--all flag) including development
branches that were never merged to master. This caused 70% false positives
from paths that only existed during development and were never published.
Example: The CamSoper/content-reorg development branch had paths like
/docs/get-started/iac/gcp/ which were restructured to /docs/iac/get-started/gcp/
before merging. These dev-only paths don't need aliases because no external
links were ever created to them.
Changed from `--all` to `origin/master` to only check published history.
Results:
- Before: 93 missing aliases (65 false positives from /docs/get-started/*)
- After: 21 missing aliases (all legitimate)
- 77% reduction in false positives
The 21 remaining are real issues including the OIDC provider files from
the May 2024 documentation revamp.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
* Add missing historical aliases for 8 files
Applied fixes from the comprehensive historical alias verification, adding
8 missing aliases that the branch-diff verification missed:
Key fixes:
- OIDC provider files (AWS, Azure, GCP, index) from May 2024 documentation
revamp where files were significantly rewritten (33-50% similarity)
- OIDC client kubernetes-eks file
- Developer portals templates file (including expected path that never existed)
- Deployments post-automation and API reference files
These were found using git's 30% similarity detection to catch low-similarity
renames (files moved and significantly rewritten in the same commit).
Also updated README.md to document the critical importance of reviewing
generated fixes for false positives before committing, including common
patterns to watch for and how to remove incorrect aliases.
Note: The original report found 21 missing aliases, but 13 were false
positives from similarity matching (Doppler/Infisical files, CLI command
pseudo-renames, etc.) and were manually removed.
Related: #16292, #16119
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
* Fix grammar error in meta_desc: 'Lean' -> 'Learn'
Co-authored-by: Cam Soper <CamSoper@users.noreply.github.com>
* Clarify prohibition on new files to specify that Markdown (.md) files must always end with a newline
---------
Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Cam Soper <CamSoper@users.noreply.github.com>1 parent a9adefa commit 8949e3a
File tree
14 files changed
+860
-34
lines changed- content/docs
- administration/access-identity/oidc-client
- deployments/deployments
- oidc
- using
- idp/developer-portals/templates
- reference/cloud-rest-api/deployments
- scripts/alias-verification
14 files changed
+860
-34
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
84 | 84 | | |
85 | 85 | | |
86 | 86 | | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
42 | 42 | | |
43 | 43 | | |
44 | 44 | | |
45 | | - | |
| 45 | + | |
46 | 46 | | |
47 | 47 | | |
48 | 48 | | |
| |||
Lines changed: 3 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
14 | 14 | | |
15 | 15 | | |
16 | 16 | | |
17 | | - | |
18 | | - | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
19 | 20 | | |
20 | 21 | | |
21 | 22 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
14 | | - | |
15 | | - | |
16 | | - | |
17 | | - | |
18 | | - | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
19 | 20 | | |
20 | 21 | | |
21 | 22 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
14 | | - | |
15 | | - | |
16 | | - | |
17 | | - | |
18 | | - | |
19 | | - | |
20 | | - | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
21 | 22 | | |
22 | 23 | | |
23 | 24 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
14 | | - | |
15 | | - | |
16 | | - | |
17 | | - | |
18 | | - | |
19 | | - | |
20 | | - | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
21 | 22 | | |
22 | 23 | | |
23 | 24 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
14 | | - | |
15 | | - | |
16 | | - | |
17 | | - | |
18 | | - | |
19 | | - | |
20 | | - | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
21 | 22 | | |
22 | 23 | | |
23 | 24 | | |
| |||
Lines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
8 | | - | |
| 8 | + | |
| 9 | + | |
9 | 10 | | |
10 | 11 | | |
11 | 12 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
| 5 | + | |
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| |||
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
| 15 | + | |
15 | 16 | | |
16 | 17 | | |
17 | 18 | | |
| |||
Lines changed: 4 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
11 | 10 | | |
12 | 11 | | |
13 | | - | |
14 | | - | |
15 | 12 | | |
16 | 13 | | |
| 14 | + | |
17 | 15 | | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
18 | 19 | | |
19 | 20 | | |
20 | 21 | | |
| |||
0 commit comments