feat: add /update-skills CRM slash command (#47)

[happysmile001]

Add Discord command to update structured skills (cSkillAttrs) with strength ratings 1-5. Members can update their own skills; Steering Committee+ can update skills for other users. New skills are merged with existing ones, preserving unchanged entries.

All 7 integration tests pass. Here's what was verified end-to-end:

Test What it verifies

1. Basic parsing "python:5, react:4, go:3" parses correctly
2. Alias normalization Node.JS→node, js→javascript, k8s→kubernetes, py→python
3. Edge cases Trailing commas, extra spaces, duplicate skills (last wins)
4. Rejection cases Empty input, missing colon, empty skill name, strength 0/6/-1/abc
5. Merge + roundtrip Existing skills preserved, new added, overlapping updated, JSON serialization roundtrips correctly
6. Full command flow Defer → search contact → fetch → merge → PUT → embed response with * markers on updated skills
7. Permission denied Member role blocked from updating others, no API call made

Summary by CodeRabbit

• New Features

  • Added an /update-skills slash command to add or merge comma-separated skill:strength entries, resolve target contacts, enforce role-based permission for updating others, and return a formatted confirmation with a CRM profile link.
  • Improved input parsing, normalization, and validation with clear user feedback and audit events.

• Error Handling

  • Handles API and unexpected errors with logging, auditing, and user-facing messages; resilient audit logging.

• Tests

  • Added extensive unit tests covering parsing, normalization, self/other updates, permission checks, merging, and error cases.

[happysmile001]

Add Discord command to update structured skills (cSkillAttrs) with strength ratings 1-5. Members can update their own skills; Steering Committee+ can update skills for other users. New skills are merged with existing ones, preserving unchanged entries.

All 7 integration tests pass. Here's what was verified end-to-end:

Test What it verifies

1. Basic parsing "python:5, react:4, go:3" parses correctly
2. Alias normalization Node.JS→node, js→javascript, k8s→kubernetes, py→python
3. Edge cases Trailing commas, extra spaces, duplicate skills (last wins)
4. Rejection cases Empty input, missing colon, empty skill name, strength 0/6/-1/abc
5. Merge + roundtrip Existing skills preserved, new added, overlapping updated, JSON serialization roundtrips correctly
6. Full command flow Defer → search contact → fetch → merge → PUT → embed response with * markers on updated skills
7. Permission denied Member role blocked from updating others, no API call made

Summary by CodeRabbit

• New Features

  • Added an /update-skills slash command to add or merge comma-separated skill:strength entries, resolve target contacts, enforce role-based permission for updating others, and return a formatted confirmation with a CRM profile link.
  • Improved input parsing, normalization, and validation with clear user feedback and audit events.

• Error Handling

  • Handles API and unexpected errors with logging, auditing, and user-facing messages; resilient audit logging.

• Tests

  • Added extensive unit tests covering parsing, normalization, self/other updates, permission checks, merging, and error cases.

[happysmile001]

Add Discord command to update structured skills (cSkillAttrs) with strength ratings 1-5. Members can update their own skills; Steering Committee+ can update skills for other users. New skills are merged with existing ones, preserving unchanged entries.

All 7 integration tests pass. Here's what was verified end-to-end:

Test What it verifies

1. Basic parsing "python:5, react:4, go:3" parses correctly
2. Alias normalization Node.JS→node, js→javascript, k8s→kubernetes, py→python
3. Edge cases Trailing commas, extra spaces, duplicate skills (last wins)
4. Rejection cases Empty input, missing colon, empty skill name, strength 0/6/-1/abc
5. Merge + roundtrip Existing skills preserved, new added, overlapping updated, JSON serialization roundtrips correctly
6. Full command flow Defer → search contact → fetch → merge → PUT → embed response with * markers on updated skills
7. Permission denied Member role blocked from updating others, no API call made

Summary by CodeRabbit

• New Features

  • Added an /update-skills slash command to add or merge comma-separated skill:strength entries, resolve target contacts, enforce role-based permission for updating others, and return a formatted confirmation with a CRM profile link.
  • Improved input parsing, normalization, and validation with clear user feedback and audit events.

• Error Handling

  • Handles API and unexpected errors with logging, auditing, and user-facing messages; resilient audit logging.

• Tests

  • Added extensive unit tests covering parsing, normalization, self/other updates, permission checks, merging, and error cases.

[happysmile001]

Add Discord command to update structured skills (cSkillAttrs) with strength ratings 1-5. Members can update their own skills; Steering Committee+ can update skills for other users. New skills are merged with existing ones, preserving unchanged entries.

All 7 integration tests pass. Here's what was verified end-to-end:

Test What it verifies

1. Basic parsing "python:5, react:4, go:3" parses correctly
2. Alias normalization Node.JS→node, js→javascript, k8s→kubernetes, py→python
3. Edge cases Trailing commas, extra spaces, duplicate skills (last wins)
4. Rejection cases Empty input, missing colon, empty skill name, strength 0/6/-1/abc
5. Merge + roundtrip Existing skills preserved, new added, overlapping updated, JSON serialization roundtrips correctly
6. Full command flow Defer → search contact → fetch → merge → PUT → embed response with * markers on updated skills
7. Permission denied Member role blocked from updating others, no API call made

Summary by CodeRabbit

• New Features

  • Added an /update-skills slash command to add or merge comma-separated skill:strength entries, resolve target contacts, enforce role-based permission for updating others, and return a formatted confirmation with a CRM profile link.
  • Improved input parsing, normalization, and validation with clear user feedback and audit events.

• Error Handling

  • Handles API and unexpected errors with logging, auditing, and user-facing messages; resilient audit logging.

• Tests

  • Added extensive unit tests covering parsing, normalization, self/other updates, permission checks, merging, and error cases.

[happysmile001]

Add Discord command to update structured skills (cSkillAttrs) with strength ratings 1-5. Members can update their own skills; Steering Committee+ can update skills for other users. New skills are merged with existing ones, preserving unchanged entries.

All 7 integration tests pass. Here's what was verified end-to-end:

Test What it verifies

1. Basic parsing "python:5, react:4, go:3" parses correctly
2. Alias normalization Node.JS→node, js→javascript, k8s→kubernetes, py→python
3. Edge cases Trailing commas, extra spaces, duplicate skills (last wins)
4. Rejection cases Empty input, missing colon, empty skill name, strength 0/6/-1/abc
5. Merge + roundtrip Existing skills preserved, new added, overlapping updated, JSON serialization roundtrips correctly
6. Full command flow Defer → search contact → fetch → merge → PUT → embed response with * markers on updated skills
7. Permission denied Member role blocked from updating others, no API call made

Summary by CodeRabbit

• New Features

  • Added an /update-skills slash command to add or merge comma-separated skill:strength entries, resolve target contacts, enforce role-based permission for updating others, and return a formatted confirmation with a CRM profile link.
  • Improved input parsing, normalization, and validation with clear user feedback and audit events.

• Error Handling

  • Handles API and unexpected errors with logging, auditing, and user-facing messages; resilient audit logging.

• Tests

  • Added extensive unit tests covering parsing, normalization, self/other updates, permission checks, merging, and error cases.

[happysmile001]

Add Discord command to update structured skills (cSkillAttrs) with strength ratings 1-5. Members can update their own skills; Steering Committee+ can update skills for other users. New skills are merged with existing ones, preserving unchanged entries.

All 7 integration tests pass. Here's what was verified end-to-end:

Test What it verifies

1. Basic parsing "python:5, react:4, go:3" parses correctly
2. Alias normalization Node.JS→node, js→javascript, k8s→kubernetes, py→python
3. Edge cases Trailing commas, extra spaces, duplicate skills (last wins)
4. Rejection cases Empty input, missing colon, empty skill name, strength 0/6/-1/abc
5. Merge + roundtrip Existing skills preserved, new added, overlapping updated, JSON serialization roundtrips correctly
6. Full command flow Defer → search contact → fetch → merge → PUT → embed response with * markers on updated skills
7. Permission denied Member role blocked from updating others, no API call made

Summary by CodeRabbit

• New Features

  • Added an /update-skills slash command to add or merge comma-separated skill:strength entries, resolve target contacts, enforce role-based permission for updating others, and return a formatted confirmation with a CRM profile link.
  • Improved input parsing, normalization, and validation with clear user feedback and audit events.

• Error Handling

  • Handles API and unexpected errors with logging, auditing, and user-facing messages; resilient audit logging.

• Tests

  • Added extensive unit tests covering parsing, normalization, self/other updates, permission checks, merging, and error cases.

[happysmile001]

Add Discord command to update structured skills (cSkillAttrs) with strength ratings 1-5. Members can update their own skills; Steering Committee+ can update skills for other users. New skills are merged with existing ones, preserving unchanged entries.

All 7 integration tests pass. Here's what was verified end-to-end:

Test What it verifies

1. Basic parsing "python:5, react:4, go:3" parses correctly
2. Alias normalization Node.JS→node, js→javascript, k8s→kubernetes, py→python
3. Edge cases Trailing commas, extra spaces, duplicate skills (last wins)
4. Rejection cases Empty input, missing colon, empty skill name, strength 0/6/-1/abc
5. Merge + roundtrip Existing skills preserved, new added, overlapping updated, JSON serialization roundtrips correctly
6. Full command flow Defer → search contact → fetch → merge → PUT → embed response with * markers on updated skills
7. Permission denied Member role blocked from updating others, no API call made

Summary by CodeRabbit

• New Features

  • Added an /update-skills slash command to add or merge comma-separated skill:strength entries, resolve target contacts, enforce role-based permission for updating others, and return a formatted confirmation with a CRM profile link.
  • Improved input parsing, normalization, and validation with clear user feedback and audit events.

• Error Handling

  • Handles API and unexpected errors with logging, auditing, and user-facing messages; resilient audit logging.

• Tests

  • Added extensive unit tests covering parsing, normalization, self/other updates, permission checks, merging, and error cases.

[happysmile001]

Add Discord command to update structured skills (cSkillAttrs) with strength ratings 1-5. Members can update their own skills; Steering Committee+ can update skills for other users. New skills are merged with existing ones, preserving unchanged entries.

All 7 integration tests pass. Here's what was verified end-to-end:

Test What it verifies

1. Basic parsing "python:5, react:4, go:3" parses correctly
2. Alias normalization Node.JS→node, js→javascript, k8s→kubernetes, py→python
3. Edge cases Trailing commas, extra spaces, duplicate skills (last wins)
4. Rejection cases Empty input, missing colon, empty skill name, strength 0/6/-1/abc
5. Merge + roundtrip Existing skills preserved, new added, overlapping updated, JSON serialization roundtrips correctly
6. Full command flow Defer → search contact → fetch → merge → PUT → embed response with * markers on updated skills
7. Permission denied Member role blocked from updating others, no API call made

Summary by CodeRabbit

• New Features

  • Added an /update-skills slash command to add or merge comma-separated skill:strength entries, resolve target contacts, enforce role-based permission for updating others, and return a formatted confirmation with a CRM profile link.
  • Improved input parsing, normalization, and validation with clear user feedback and audit events.

• Error Handling

  • Handles API and unexpected errors with logging, auditing, and user-facing messages; resilient audit logging.

• Tests

  • Added extensive unit tests covering parsing, normalization, self/other updates, permission checks, merging, and error cases.