Skip to content

Generate the canonical sitemap and show GitHub stars - #37

Merged
rclanan merged 1 commit into
mainfrom
fix/marketing-seo-backlog
Oct 12, 2026
Merged

rclanan merged 1 commit into
mainfrom
fix/marketing-seo-backlog

Conversation

@rclanan

@rclanan rclanan commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

The marketing sitemap was maintained separately from the rendered pages, leaving newer indexable pages out and including pages marked noindex. Build the sitemap from the pages and their canonical URLs, exclude redirects and noindex pages, and emit identical copies at both public/sitemap.xml and the deployed root. Add the live GitHub stars badge to the homepage hero.

Validation: the complete CSS/pages/site build passes; all 104 internal links pass; generated pages are in sync. An independent build fixture confirms a new indexable page is automatically included and a new noindex page is excluded. The XML contains 43 unique canonical URLs. Orca verifies the badge loads and the page has no horizontal overflow at desktop and 375px widths.

The optional external link check encountered a 503 for the existing GitHub CONFORMANCE_TESTING.md link; the connected browser loads that exact page successfully and the GitHub contents API confirms it exists. No external link was changed to conceal that transport failure.

Related to SaaSy-Solutions/mockforge#454. The Postman comparison and robots sitemap reference already exist. Google Search Console verification and submission remain tracked separately; this PR does not close the issue.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@codedig-ai

codedig-ai Bot commented Oct 12, 2026

Copy link
Copy Markdown

🟡 CodeDig PR Analysis — Medium Risk (Score: 99/100)

🤖 Summary

This PR is low risk (1 findings, 1 files changed).

Summary

Metric Value
Files Changed 1
Total Findings ⚠️ 1
Critical 0
High 0
Public API Changes 0
New PII Flows 0
Complexity Spikes ⚠️ 1
Co-change Risks 0

Risk Budget: 1/100 consumed (green status)

| Coverage data | not available |
⚠️ Key Findings

  • [medium] High complexity in scripts/build-pages.js — scripts/build-pages.js

✅ Recommendations

  • Consider refactoring to reduce complexity. Extract methods, simplify conditionals, or use polymorphism to replace complex branching.
🧪 Test Coverage Delta (no data)

Coverage data not available for this repo — enable via Settings → Coverage

Detailed Findings

[medium] High complexity in scripts/build-pages.js

Cyclomatic complexity spike detected: scripts/build-pages.js (complexity: 12, delta: +6)
Affected files: scripts/build-pages.js
Recommendation: Consider refactoring to reduce complexity. Extract methods, simplify conditionals, or use polymorphism to replace complex branching.

→ View full report


Powered by CodeDig

1 similar comment
@codedig-ai

codedig-ai Bot commented Oct 12, 2026

Copy link
Copy Markdown

🟡 CodeDig PR Analysis — Medium Risk (Score: 99/100)

🤖 Summary

This PR is low risk (1 findings, 1 files changed).

Summary

Metric Value
Files Changed 1
Total Findings ⚠️ 1
Critical 0
High 0
Public API Changes 0
New PII Flows 0
Complexity Spikes ⚠️ 1
Co-change Risks 0

Risk Budget: 1/100 consumed (green status)

| Coverage data | not available |
⚠️ Key Findings

  • [medium] High complexity in scripts/build-pages.js — scripts/build-pages.js

✅ Recommendations

  • Consider refactoring to reduce complexity. Extract methods, simplify conditionals, or use polymorphism to replace complex branching.
🧪 Test Coverage Delta (no data)

Coverage data not available for this repo — enable via Settings → Coverage

Detailed Findings

[medium] High complexity in scripts/build-pages.js

Cyclomatic complexity spike detected: scripts/build-pages.js (complexity: 12, delta: +6)
Affected files: scripts/build-pages.js
Recommendation: Consider refactoring to reduce complexity. Extract methods, simplify conditionals, or use polymorphism to replace complex branching.

→ View full report


Powered by CodeDig

@rclanan
rclanan merged commit 1948b83 into main Oct 12, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant