Skip to content

Enhance SEO, update assets, and address review comments#38

Merged
sumitsahoo merged 9 commits intomainfrom
dev
Apr 7, 2026
Merged

Enhance SEO, update assets, and address review comments#38
sumitsahoo merged 9 commits intomainfrom
dev

Conversation

@sumitsahoo
Copy link
Copy Markdown
Owner

This pull request focuses on improving SEO, updating dependencies, and making minor documentation and configuration changes for the CloakPDF project. The most significant updates include enhanced metadata for search and social platforms, dependency upgrades for better stability and security, and the addition of standard SEO files.

SEO and discoverability improvements:

  • Enhanced SEO metadata in index.html, including updated descriptions, keywords, canonical URL, Open Graph, Twitter Card tags, and structured data for better search engine and social platform visibility. The page title was also updated for clarity and keyword targeting.
  • Added public/robots.txt to allow all user agents and reference the sitemap, improving search engine indexing.
  • Added public/sitemap.xml to provide search engines with a structured list of site URLs.

Dependency and package management updates:

  • Upgraded @voidzero-dev/vite-plus-core from version 0.1.15 to 0.1.16 in package.json and pnpm-lock.yaml, along with related dependency updates and removals (including updates to lru-cache and electron-to-chromium). This ensures the project uses the latest features and security patches. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12]

Documentation and configuration improvements:

  • Updated the main logo and description in README.md for better branding and clarity, and switched the demo link to use HTTPS.
  • Clarified the description of third-party dependency monitoring in SECURITY.md for improved transparency.

Copilot AI review requested due to automatic review settings April 7, 2026 08:18
@sumitsahoo sumitsahoo merged commit d12f74e into main Apr 7, 2026
6 checks passed
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Improves CloakPDF’s SEO/discoverability and refreshes branding assets, alongside a small dependency/toolchain upgrade and minor documentation/security policy edits.

Changes:

  • Expanded SEO metadata in index.html (canonical, Open Graph, Twitter Card, and JSON-LD structured data).
  • Added standard SEO crawl/indexing files: public/robots.txt and public/sitemap.xml.
  • Updated README branding/link and upgraded @voidzero-dev/vite-plus-core (with corresponding lockfile changes).

Reviewed changes

Copilot reviewed 6 out of 11 changed files in this pull request and generated no comments.

Show a summary per file
File Description
SECURITY.md Clarifies dependency monitoring language in the security model section.
README.md Updates hero image and switches demo URL to HTTPS.
public/sitemap.xml Adds a basic sitemap for search engine discovery.
public/robots.txt Allows crawling and points bots to the sitemap URL.
public/icons/og-image.svg Adds an SVG OG/branding asset (used in README).
pnpm-lock.yaml Updates lockfile entries for the dependency upgrade(s).
package.json Bumps @voidzero-dev/vite-plus-core version (but leaves a mismatched vite alias).
index.html Enhances SEO and social sharing metadata and updates the page title.
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (1)

package.json:33

  • @voidzero-dev/vite-plus-core was bumped to ^0.1.16, but the vite alias still points to npm:@voidzero-dev/vite-plus-core@^0.1.15. This can lead to mismatched toolchain versions (and makes the lockfile harder to reason about). Update the vite alias to match the bumped version (or rely solely on the existing pnpm.overrides.vite).
    "@voidzero-dev/vite-plus-core": "^0.1.16",
    "tailwindcss": "^4.2.2",
    "typescript": "^6.0.2",
    "vite": "npm:@voidzero-dev/vite-plus-core@^0.1.15",
    "vite-plugin-pwa": "^1.2.0",

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.

2 participants