Skip to content

Conversation

@Tigeredgar
Copy link

@Tigeredgar Tigeredgar commented Jan 30, 2026

Description

Screencast

Checklist

@raycastbot
Copy link
Collaborator

Congratulations on your new Raycast extension! 🚀

We're currently experiencing a high volume of incoming requests. As a result, the initial review may take up to 10-15 business days.

Once the PR is approved and merged, the extension will be available on our Store.

@Tigeredgar Tigeredgar marked this pull request as ready for review January 30, 2026 15:52
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 30, 2026

Greptile Overview

Greptile Summary

This PR adds a new Okta Manager extension that allows users to manage Okta applications, users, and groups through Raycast. The extension provides search functionality and editing capabilities for OIDC redirect URIs, SAML settings, user profiles, and group metadata.

Key Changes:

  • Implements 4 view commands: manage environments, search apps, search users, search groups
  • API client with proper authentication using SSWS tokens stored in local config
  • Environment management system to switch between multiple Okta instances
  • OIDC and SAML configuration editors for applications

Critical Issues:

  • Missing required metadata/ folder with Raycast-styled screenshots (all view commands require this)
  • Uses confirm() function which doesn't exist in Raycast environment

Code Quality:

  • Follows Raycast coding standards (Prettier config, ESLint, changelog format)
  • Proper error handling and loading states throughout
  • Good API abstraction with type definitions
  • Appropriate use of debouncing for search operations

Confidence Score: 2/5

  • Cannot be merged due to missing metadata folder and runtime error with confirm() function
  • Two blocking issues prevent merge: (1) missing required metadata folder with screenshots for view commands, which is mandatory per repository guidelines, and (2) use of confirm() function that doesn't exist in Raycast API causing runtime errors. Code quality is otherwise excellent with proper architecture and error handling.
  • package.json needs metadata folder added, manage-environments.tsx needs confirm() replaced with Raycast API

Important Files Changed

Filename Overview
extensions/okta-app-manager/package.json Adds Okta Manager extension with 4 view commands; missing required metadata folder with screenshots
extensions/okta-app-manager/src/api/client.ts Implements Okta API client with proper authentication and error handling
extensions/okta-app-manager/src/api/config.ts Manages environment configuration storage using fs and environment.supportPath
extensions/okta-app-manager/src/manage-environments.tsx Environment management UI with create/edit/delete actions; uses invalid confirm() function
extensions/okta-app-manager/src/search-apps.tsx App search with OIDC/SAML management actions and proper debouncing

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.

5 files reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

Comment on lines +16 to +41
"commands": [
{
"name": "manage-environments",
"title": "Manage Environments",
"description": "Switch between Okta environments",
"mode": "view"
},
{
"name": "search-apps",
"title": "Search Apps",
"description": "Search and manage Okta applications",
"mode": "view"
},
{
"name": "search-users",
"title": "Search Users",
"description": "Search and manage Okta users",
"mode": "view"
},
{
"name": "search-groups",
"title": "Search Groups",
"description": "Search and manage Okta groups",
"mode": "view"
}
],
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing metadata/ folder with screenshots. All view-type commands require Raycast-styled screenshots in a metadata/ folder for the store.

Context Used: Rule from dashboard - What: Extensions with view-type commands must include a metadata/ folder containing Raycast-styled... (source)

}

async function handleDelete(name: string) {
if (confirm(`Delete environment "${name}"?`)) {
Copy link
Contributor

Choose a reason for hiding this comment

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

confirm() is not available in Raycast. Use Action with confirmation or Alert API instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants