Skip to content

docs: Add Incy client#349

Open
0-Kutya-0 wants to merge 4 commits intoremnawave:mainfrom
0-Kutya-0:Incy-app
Open

docs: Add Incy client#349
0-Kutya-0 wants to merge 4 commits intoremnawave:mainfrom
0-Kutya-0:Incy-app

Conversation

@0-Kutya-0
Copy link
Contributor

Add Incy client(HWID enabled by default)

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 18, 2026

Greptile Summary

This PR adds the Incy VPN client to the documentation site, including a new entry in src/data/clients.ts, an SVG logo, and a listing in the HWID device limit documentation. The changes follow existing patterns and conventions.

  • Missing HWID badge: The client entry in clients.ts is missing badges: { hwid: true } even though the PR description states HWID is enabled by default and the HWID docs list it without a "disabled by default" note. This should be added for the HWID badge to display in the UI.
  • Docs and logo file changes look correct and consistent with existing entries.

Confidence Score: 3/5

  • Low-risk docs/data PR but has a missing HWID badge that contradicts the stated intent.
  • Score of 3 because the PR is a simple data addition, but the missing badges: { hwid: true } in the client entry contradicts both the PR description and the HWID documentation entry. This means the HWID badge won't render for Incy in the UI, which is likely unintentional.
  • src/data/clients.ts needs the badges: { hwid: true } property added to the Incy client entry.

Important Files Changed

Filename Overview
src/data/clients.ts New Incy client entry added but missing badges: { hwid: true } despite PR description stating HWID is enabled by default. Also adds a trailing newline to fix missing EOF newline.
docs/features/hwid-device-limit.md Correctly adds Incy to the HWID-supported applications list without a "disabled by default" note, consistent with PR description.
static/clients/logo/incy-dark.svg Simple SVG placeholder logo with "IN" text on a rounded rectangle. Follows naming convention (incy-dark.svg).

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[clients.ts] -->|Defines client data| B[CLIENTS array]
    B -->|getClientsByPlatform| C[iOS clients list]
    B -->|getClientsByPlatform| D[macOS clients list]
    B -->|badges.hwid check| E{HWID badge shown?}
    E -->|hwid: true| F[✅ Badge displayed]
    E -->|missing badges| G[❌ No badge]
    H[hwid-device-limit.md] -->|Lists Incy| I[HWID supported apps]
    J[incy-dark.svg] -->|Referenced by| A
Loading

Last reviewed commit: d7bd845

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

3 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

docs: mark Incy as supporting HWID on the Clients page
@kastov
Copy link
Contributor

kastov commented Feb 18, 2026

Hi there!

Is that your project?

@0-Kutya-0
Copy link
Contributor Author

Hi there!

Is that your project?

No, but from first impressions it works better than Happ, at least it consumes less battery power

@JustYay
Copy link

JustYay commented Feb 19, 2026

As the creator of INCY, I’d like to say thank you for the PR 🙌

At the moment, the application is not yet ready for Production.
The first Production version is currently under review for both iOS (App Store) and Android (Google Play).

I will let you know once we are ready to move forward with the Production release.

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.

3 participants