Skip to content

added cognito-to-descope migration skill#10

Open
mrunankpawar wants to merge 7 commits into
mainfrom
cognito-to-descope
Open

added cognito-to-descope migration skill#10
mrunankpawar wants to merge 7 commits into
mainfrom
cognito-to-descope

Conversation

@mrunankpawar

@mrunankpawar mrunankpawar commented Apr 30, 2026

Copy link
Copy Markdown
Collaborator

Description

This skill will be helpful for customers and prospects to help/guide them migrate from Cognito to Descope with any agent of their choice.

@shuni-bot-dev

shuni-bot-dev Bot commented Apr 30, 2026

Copy link
Copy Markdown
Contributor

🐕 Review complete — View session on Shuni Portal 🐾

@shuni-bot-dev shuni-bot-dev Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🐕 Shuni's Review

Adds a new cognito-to-descope migration skill (~2,800 lines of guidance + reference docs).

Sniffed out 4 issues:

  • 1 🟠 HIGH: plugin.json version not bumped per CLAUDE.md (new skill = minor bump → 1.2.0)
  • 3 🟡 MEDIUM: incorrect Node SDK method name (management.user.logout doesn't exist), wrong Python kwarg (logout_user(user_id=...) will TypeError), JWT payload decoded with 'base64' instead of 'base64url'

The 🟠 HIGH is a clear project-rule violation. The 🟡 MEDIUMs in IMPLEMENTATION-NOTES-DIST.md will lead users to write code that doesn't run — worth fixing before this lands so it can't propagate into customer migrations. Needs a bath!

Note on plugin.json: per CLAUDE.md, every PR must bump .claude-plugin/plugin.json. A new skill is a minor bump. Current version on this branch is still 1.1.1 (same as main); it should be 1.2.0.

Comment thread skills/cognito-to-descope/references/IMPLEMENTATION-NOTES-DIST.md Outdated
Comment thread skills/cognito-to-descope/references/IMPLEMENTATION-NOTES-DIST.md Outdated
Comment thread skills/cognito-to-descope/references/IMPLEMENTATION-NOTES-DIST.md Outdated
Comment thread skills/cognito-to-descope/SKILL.md Outdated
Comment thread skills/cognito-to-descope/references/flows-widgets-console.md Outdated
Comment thread skills/cognito-to-descope/references/flows-widgets-console.md Outdated
Comment thread skills/cognito-to-descope/references/flows-widgets-console.md Outdated
Comment thread skills/cognito-to-descope/references/flows-widgets-console.md Outdated
Comment thread skills/cognito-to-descope/SKILL.md
Comment thread skills/cognito-to-descope/references/flows-widgets-console.md Outdated
Comment thread skills/cognito-to-descope/references/flows-widgets-console.md
Comment thread skills/cognito-to-descope/references/flows-widgets-console.md Outdated
Comment thread skills/cognito-to-descope/references/flows-widgets-console.md Outdated
Comment thread skills/cognito-to-descope/references/flows-widgets-console.md Outdated
Comment thread skills/cognito-to-descope/references/flows-widgets-console.md Outdated
Comment thread skills/cognito-to-descope/references/flows-widgets-console.md Outdated
Comment thread skills/cognito-to-descope/references/flows-widgets-console.md
Comment thread skills/cognito-to-descope/references/flows-widgets-console.md Outdated
Comment thread skills/cognito-to-descope/references/implementation-guide.md Outdated
Comment thread skills/cognito-to-descope/references/implementation-guide.md Outdated
Comment thread skills/cognito-to-descope/references/implementation-guide.md Outdated
Comment thread skills/cognito-to-descope/references/implementation-guide.md Outdated
Comment thread skills/cognito-to-descope/references/implementation-guide.md Outdated
Comment thread skills/cognito-to-descope/references/implementation-guide.md Outdated
Comment thread skills/cognito-to-descope/references/implementation-guide.md Outdated
Comment thread skills/cognito-to-descope/references/implementation-guide.md Outdated
Comment thread skills/cognito-to-descope/references/implementation-guide.md
Comment thread skills/cognito-to-descope/references/migration-plan-template.md
@mrunankpawar mrunankpawar requested a review from gaokevin1 May 2, 2026 23:19
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