Skip to content

feat(skills): admin skills management parity (#51)#52

Merged
mpge merged 5 commits into
mainfrom
feat/admin-skills-management
May 17, 2026
Merged

feat(skills): admin skills management parity (#51)#52
mpge merged 5 commits into
mainfrom
feat/admin-skills-management

Conversation

@mpge
Copy link
Copy Markdown
Member

@mpge mpge commented May 17, 2026

Summary

  • Adds SkillRoutingTag / SkillRoutingDepartment, optional Skill.description, and AgentSkill proficiency default 3 with ORM validators plus a DB check constraint (migration 0022).
  • Admin Skills index/edit/create match the shared Escalated contract: annotated counts, form props (skill, availableAgents, availableTags, availableDepartments), store/update/destroy routes, and transactional saves for routing and agent rows.
  • SkillRoutingService uses explicit tag or department overlap, requires AgentSkill rows for all matched skills, and orders by sum of proficiency then open ticket load.

Closes #51

Test plan

  • python -m pytest
  • ruff check escalated tests
  • Smoke admin Skills pages in a host app with Inertia

mpge and others added 5 commits May 17, 2026 12:53
Introduces SkillRoutingTag/SkillRoutingDepartment, skill.description, and AgentSkill proficiency default 3 with a 1-5 check constraint.

Co-authored-by: Cursor <cursoragent@cursor.com>
Index counts, form context (camelCase props per shared UI), store/update/destroy routes, transactional saves, and SkillRoutingService overlap + intersection logic ordered by proficiency sum then load.

Co-authored-by: Cursor <cursoragent@cursor.com>
Covers store/update JSON, DELETE destroy, explicit tag routing vs name match, ordering, departments, and form user-filter Q construction.

Co-authored-by: Cursor <cursoragent@cursor.com>
FOSSA fails in CI because secrets.FOSSA_API_KEY isn't configured. Per the
2026-05-17 portfolio sweep, remove the workflow on every backend where it's
red so the suite can go green. To restore: re-add this file (or copy from
escalated-laravel / escalated where FOSSA is wired up correctly) and set
the FOSSA_API_KEY secret on the repo.
@mpge mpge merged commit 8639c47 into main May 17, 2026
9 checks passed
@mpge mpge deleted the feat/admin-skills-management branch May 17, 2026 17:06
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.

skills management parity: bring admin skills UI to feature parity with the NestJS reference

1 participant