parse typeid #17067

iloveitaly · 2025-02-14T21:42:35Z

Description

Add typeid parsing support.

Screencast

Checklist

I read the extension guidelines
I read the documentation about publishing
I ran npm run build and tested this distribution build in Raycast
I checked that files in the assets folder are used by the extension itself
I checked that assets used by the README are placed outside of the metadata folder

raycastbot · 2025-02-14T21:42:54Z

Thank you for your contribution! 🎉

🔔 @jmaeso @GastroGeek @pernielsentikaer @LunaticMuch @hmarr @harsh-devx @lukas-runge you might want to have a look.

You can use this guide to learn how to check out the Pull Request locally in order to test it.

You can expect an initial review within five business days.

LitoMore · 2025-02-15T15:41:58Z

extensions/uuid-generator/package.json

+    "@eslint/compat": "^1.2.6",
+    "@eslint/eslintrc": "^3.2.0",
+    "@eslint/js": "^9.20.0",
+    "@raycast/eslint-config": "^1.0.11",
+    "@types/node": "^22.13.4",
    "@types/uuid": "^10.0.0",
-    "eslint": "^8.45.0",
-    "eslint-plugin-import": "^2.27.5",
-    "react-devtools": "^4.28.0",
-    "typescript": "^5.1.6"
+    "@typescript-eslint/eslint-plugin": "^8.24.0",
+    "eslint": "^9.20.1",
+    "eslint-plugin-import": "^2.31.0",


Just trying to keep packages up to date to avoid weird old compatibility issues

AFAIK, @raycast/api does not support ESLint 9 yet.

ah, interesting, that's why this was harder than I expected :)

pernielsentikaer

Why did we remove .eslintrc.json? I should probably still exists with

{
  "root": true,
  "extends": ["@raycast"]
}

pernielsentikaer · 2025-02-24T12:56:39Z

extensions/uuid-generator/eslint.config.mjs

@@ -0,0 +1,43 @@
+import typescriptEslint from "@typescript-eslint/eslint-plugin";


Do we need this file at all?

I think so? Didn't try to remove it, but this definitely the way configuration is done in the latest ESLint version.

We don't need this. You can refer to other extensions' configurations since you changed ESLint back to version 8.

greptile-apps

PR Summary

This PR adds TypeID parsing functionality to the UUID Generator extension, allowing users to decode TypeIDs back into UUIDs with copy/paste support.

New parseTypeID.ts could use showFailureToast from @raycast/utils instead of manual showToast with failure style
launchCommand in parseTypeID.ts should be wrapped in try/catch block for better error handling
Consider adding subtitle "UUID Generator" to commands in package.json since there are multiple commands
Ensure metadata folder exists with screenshots since this extension has view commands
Changelog entry uses {PR_MERGE_DATE} correctly for automatic date replacement

_{💡 (1/5) You can manually trigger the bot by mentioning @greptileai in a comment!}

_{6 file(s) reviewed, 3 comment(s)}
_{Edit PR Review Bot Settings | Greptile}

greptile-apps · 2025-02-24T12:54:35Z

extensions/uuid-generator/eslint.config.mjs

+    rules: {
+        "@typescript-eslint/no-unused-vars": [
+            "warn",
+            {
+                "argsIgnorePattern": "^_",
+                "varsIgnorePattern": "^_",
+                "caughtErrorsIgnorePattern": "^_"
+            }
+    ],
+    },


style: Indentation is inconsistent in the rules section (extra spaces before @typescript-eslint/no-unused-vars and misaligned closing bracket)

It's better not to bring some of your preferred rules to this project. Keeping all extension code styles consistent is good for the community.

Agree, we should use the default esline config only:

{ "printWidth": 120, "singleQuote": false }

greptile-apps · 2025-02-24T12:54:36Z

extensions/uuid-generator/package.json

+      "name": "parseTypeID",
+      "title": "Parse TypeIDs",
+      "description": "Parse TypeID and copy resulting UUID to the clipboard",


style: Consider adding a subtitle 'UUID Generator' to maintain consistency with other commands in the extension

greptile-apps · 2025-02-24T12:55:37Z

package.json

@@ -1,5 +1,6 @@
 {
  "devDependencies": {
+    "@typescript-eslint/eslint-plugin": "^8.24.0",


style: Consider pinning exact version instead of using ^ to ensure consistent builds

You could ignore this one. Just remove this plugin and use Raycast's default configuration about ESLint.

iloveitaly · 2025-02-25T20:07:01Z

I don't have much more time to put into this. I was more ambitious than I should have been with the eslint upgrades and such, but we can ignore that and fix it up later, it would be great to get this merged. I don't have the time to fix that up.

iloveitaly added 2 commits February 14, 2025 14:28

build: update packages

524dcf8

[uuid-generator] feat: add parse typeid

de72543

iloveitaly requested a review from pernielsentikaer as a code owner February 14, 2025 21:42

raycastbot added extension fix / improvement Label for PRs with extension's fix improvements extension: uuid-generator Issues related to the uuid-generator extension labels Feb 14, 2025

iloveitaly added 5 commits February 14, 2025 14:44

docs: add changelog

bfe6793

style: fix linting

cbec60a

build: migrate to new eslint

28a129f

build: add missing package

11e3559

build: add package

8e1a315

LitoMore reviewed Feb 15, 2025

View reviewed changes

build: fix linting errors

241a4f5

pernielsentikaer self-assigned this Feb 24, 2025

pernielsentikaer reviewed Feb 24, 2025

View reviewed changes

greptile-apps bot reviewed Feb 24, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

parse typeid #17067

parse typeid #17067

iloveitaly commented Feb 14, 2025 •

edited

Loading

raycastbot commented Feb 14, 2025 •

edited

Loading

LitoMore Feb 15, 2025

iloveitaly Feb 16, 2025

LitoMore Feb 25, 2025 •

edited

Loading

iloveitaly Feb 25, 2025

pernielsentikaer left a comment

pernielsentikaer Feb 24, 2025

iloveitaly Feb 25, 2025

LitoMore Feb 25, 2025 •

edited

Loading

greptile-apps bot left a comment

greptile-apps bot Feb 24, 2025

LitoMore Feb 25, 2025

pernielsentikaer Feb 25, 2025

greptile-apps bot Feb 24, 2025

greptile-apps bot Feb 24, 2025

LitoMore Feb 25, 2025 •

edited

Loading

pernielsentikaer Feb 25, 2025

iloveitaly commented Feb 25, 2025

		@@ -0,0 +1,43 @@
		import typescriptEslint from "@typescript-eslint/eslint-plugin";

parse typeid #17067

Are you sure you want to change the base?

parse typeid #17067

Conversation

iloveitaly commented Feb 14, 2025 • edited Loading

Description

Screencast

Checklist

raycastbot commented Feb 14, 2025 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

LitoMore Feb 25, 2025 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

pernielsentikaer left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

LitoMore Feb 25, 2025 • edited Loading

Choose a reason for hiding this comment

greptile-apps bot left a comment

Choose a reason for hiding this comment

PR Summary

greptile-apps bot Feb 24, 2025

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

greptile-apps bot Feb 24, 2025

Choose a reason for hiding this comment

greptile-apps bot Feb 24, 2025

Choose a reason for hiding this comment

LitoMore Feb 25, 2025 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

iloveitaly commented Feb 25, 2025

iloveitaly commented Feb 14, 2025 •

edited

Loading

raycastbot commented Feb 14, 2025 •

edited

Loading

LitoMore Feb 25, 2025 •

edited

Loading

LitoMore Feb 25, 2025 •

edited

Loading

LitoMore Feb 25, 2025 •

edited

Loading