Skip to content

fix(public): refresh sitemap lastmod dates to real page changes - #75

Closed
nish3451 wants to merge 19 commits into
mainfrom
fix/lane1-public-sitemap-lastmod
Closed

fix(public): refresh sitemap lastmod dates to real page changes#75
nish3451 wants to merge 19 commits into
mainfrom
fix/lane1-public-sitemap-lastmod

Conversation

@nish3451

@nish3451 nish3451 commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

What

Refreshes public/sitemap.xml so every one of the 12 URLs carries an accurate lastmod, sourced from the last git commit that touched each page's HTML.

  • Stale dates fixed: 5 URLs previously claimed 2026-05-20 (predates the site's June 2026 SEO bundle).
  • Missing dates added: 7 URLs (privacy, privacy-choices, terms, promptly/support, promptly/privacy, drishti/support, drishti/privacy) had no lastmod at all.
  • Kept the existing W3C YYYY-MM-DD format; no other sitemap fields changed.

Source of truth (git history on main)

URL Last touching commit Date
/ 4202d54 fix(public): disambiguate Tiny Studio brands (#29) 2026-08-09
all 11 others a0d1de5 fix(public): replace launch-prep footer copy (#35) 2026-08-10

The sitemap is hand-maintained (created in commit #4, never refreshed since); no generator script exists to update instead.

Validation

  • python3 xml.etree.ElementTree parse: well-formed, 12 <url> entries, every entry has <lastmod>.

Summary by CodeRabbit

  • Documentation
    • Updated sitemap modification dates to reflect the latest page updates.
    • Added formatted update timestamps for privacy, terms, support, and related pages.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@coderabbitai

coderabbitai Bot commented Aug 11, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 7498e9a9-48d3-445b-9a1f-6febe4ca1297

📥 Commits

Reviewing files that changed from the base of the PR and between b2a58e0 and 9f9b627.

📒 Files selected for processing (1)
  • public/sitemap.xml

📝 Walkthrough

Walkthrough

The sitemap updates existing lastmod dates and adds formatted lastmod elements to Privacy, terms, Promptly, and Drishti entries.

Changes

Sitemap metadata

Layer / File(s) Summary
Update sitemap modification dates
public/sitemap.xml
The sitemap uses August 9 or 10, 2026 for updated entries and adds missing lastmod elements.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

  • nish3451/TinyStudio.io#36: Both changes modify public/sitemap.xml; this PR updates lastmod metadata, while the related PR adds URLs and validation tests.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the sitemap lastmod date updates, which are the main changes in the pull request.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/lane1-public-sitemap-lastmod

Comment @coderabbitai help to get the list of available commands.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e7f8127028

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread public/sitemap.xml
</url>
<url>
<loc>https://tinystudio.in/privacy/</loc>
<lastmod>2026-08-10</lastmod>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Record the August 11 trust-page updates

When crawlers consume this sitemap, /privacy/, /terms/, and /drishti/privacy/ are reported as last modified on August 10 even though ancestor commit 62a5dfe changed each page's HTML on August 11 to repair its heading hierarchy. This contradicts the commit's HTML-history source of truth and leaves three newly added lastmod values stale immediately; update these entries to 2026-08-11.

Useful? React with 👍 / 👎.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 32d178a874

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread public/sitemap.xml
</url>
<url>
<loc>https://tinystudio.in/promptly/privacy/</loc>
<lastmod>2026-08-10</lastmod>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Set Promptly privacy lastmod to August 12

When crawlers consume this sitemap, /promptly/privacy/ is reported as last modified on August 10, but the reviewed commit's parent 6674aec changed that page's HTML on August 12 by updating its description metadata and JSON-LD. Since these values are intended to reflect the latest HTML-touching commit, set this entry to 2026-08-12.

Useful? React with 👍 / 👎.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@nish3451

Copy link
Copy Markdown
Collaborator Author

Closing as landed — main already has everything this PR set out to do, and merging it now would make the sitemap less accurate.

This PR's goal was to give every URL a real lastmod. On current main all 12 URLs already carry one, including the six that previously had none (/privacy/, /privacy-choices/, /terms/, /promptly/support/, /promptly/privacy/, /drishti/support/, /drishti/privacy/).

The only remaining delta is the conflict itself, and it points the wrong way: main has 2026-08-11 for /privacy/, /terms/ and /drishti/privacy/, while this branch still says 2026-08-10. Merging would roll those three dates backwards. Those pages also change again in #186, so their real lastmod is later still.

Nothing to rebase here. If sitemap dates drift again, a fresh PR against current main is the right fix.

@nish3451 nish3451 closed this Aug 19, 2026
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