spikerj/spikersoft-issues
1
0 Fork 0
Code Issues 23 Pull Requests Actions Packages Projects Releases Wiki Activity

Skill awards can trigger fund payouts (server skill-awarded activity + earned-skill-count provider) #874

New Issue
Closed
opened 2026-07-27 14:30:13 +00:00 by spikerj · 2 comments
spikerj commented 2026-07-27 14:30:13 +00:00
Owner
Copy link
Copy source

Part of #868 (PR-7, backend).

Today AwardSkillCommandHandler writes the EarnedSkill and returns — no server UserActivity is emitted, and the registry's skill-earned action is a client breadcrumb the RewardEngine deliberately ignores (KnowledgeDomainRegistry.cs:799). So no staff rule can pay out for earning a skill.

  • Emit a server-side UserActivity (source server, domain Learning, new action skill-awarded, metadata: skillDefinitionId, tier, interestAreaId) from AwardSkillCommandHandler after the insert. Award-only — nothing on revoke.
  • Register skill-awarded in KnowledgeDomainRegistry as a trustworthy server action (kept distinct from the skill-earned breadcrumb).
  • New IRewardStateProvider key earned-skill-count (pattern: wishlist-country-count etc.) enabling threshold state conditions ("has ≥N skills / ≥N gold skills").
  • Tests: activity emitted on award; RewardEngine matches a skill-awarded rule end-to-end (completion + FundEarning donation); metadata condition on tier works.

Result: staff can create "earn a gold skill → $X toward your dream destination" rules in the existing admin → sponsorship-management → Location funds page.

Part of #868 (PR-7, backend). Today `AwardSkillCommandHandler` writes the `EarnedSkill` and returns — no server `UserActivity` is emitted, and the registry's `skill-earned` action is a client breadcrumb the RewardEngine deliberately ignores (`KnowledgeDomainRegistry.cs:799`). So no staff rule can pay out for earning a skill. - Emit a server-side `UserActivity` (source `server`, domain `Learning`, new action `skill-awarded`, metadata: `skillDefinitionId`, `tier`, `interestAreaId`) from `AwardSkillCommandHandler` after the insert. Award-only — nothing on revoke. - Register `skill-awarded` in `KnowledgeDomainRegistry` as a trustworthy server action (kept distinct from the `skill-earned` breadcrumb). - New `IRewardStateProvider` key `earned-skill-count` (pattern: `wishlist-country-count` etc.) enabling threshold state conditions ("has ≥N skills / ≥N gold skills"). - Tests: activity emitted on award; RewardEngine matches a `skill-awarded` rule end-to-end (completion + FundEarning donation); metadata condition on `tier` works. Result: staff can create "earn a gold skill → $X toward your dream destination" rules in the existing admin → sponsorship-management → Location funds page.
spikerj commented 2026-07-29 15:31:35 +00:00
Author
Owner
Copy link
Copy source

Audited against origin/master — implementation is complete; the required tests are not. Staying open on tests only. (The standalone PR #496 was closed unmerged; the work landed via consolidated backend PR #497.)

Landed

  • SpikerSoft.Business/Domain/Skills/SkillAwardedActivity.cs:20-41 — domain learning.skills, action skill-awarded, Source = "server", all three metadata keys (skillDefinitionId, tier, interestAreaId).
  • Emitted post-insert on both award paths: AwardSkillCommandHandler.cs:57 and CheckAutoAwardsCommandHandler.cs:116. Revoke correctly emits nothing.
  • KnowledgeDomainRegistry.cs:577 — new("skill-awarded", "Skill Awarded", "Learning", false), server-trustworthy, kept distinct from the client-side skill-earned breadcrumb at :820.
  • Provider IRewardStateProvider.cs:51-58 Key => "earned-skill-count", DI-registered at ServiceCollectionExtensions.cs:625.

Missing — 2 of the 3 required test items, plus provider coverage:

  1. No RewardEngine end-to-end test that a skill-awarded rule matches and produces a completion + FundEarning. git grep -rn "skill-awarded|SkillAwardedActivity" origin/master -- '*Tests*' hits only AwardSkillCommandHandlerTests.cs and CheckAutoAwardsCommandHandlerTests.cs — nothing in SpikerSoft.Business.Tests/Domain/Funds/RewardEngineMatchingTests.cs.
  2. No metadata-condition-on-tier test — git grep -rn '"tier"' origin/master -- 'SpikerSoft.Business.Tests/Domain/Funds/' returns nothing.
  3. No coverage for EarnedSkillCountProvider — git grep -rn "earned-skill-count|EarnedSkillCountProvider" origin/master -- '*Tests*' returns nothing.

Worth weighting this properly: the untested path disburses real money, and the epic's manual "rewards smoke test" is also unrecorded, so as far as the tracker shows the payout path has been verified neither in automation nor in production. I'd treat the two engine tests as blocking rather than nice-to-have.

Audited against `origin/master` — **implementation is complete; the required tests are not.** Staying open on tests only. (The standalone PR #496 was closed unmerged; the work landed via consolidated backend PR #497.) **Landed** - `SpikerSoft.Business/Domain/Skills/SkillAwardedActivity.cs:20-41` — domain `learning.skills`, action `skill-awarded`, `Source = "server"`, all three metadata keys (`skillDefinitionId`, `tier`, `interestAreaId`). - Emitted post-insert on both award paths: `AwardSkillCommandHandler.cs:57` and `CheckAutoAwardsCommandHandler.cs:116`. Revoke correctly emits nothing. - `KnowledgeDomainRegistry.cs:577` — `new("skill-awarded", "Skill Awarded", "Learning", false)`, server-trustworthy, kept distinct from the client-side `skill-earned` breadcrumb at `:820`. - Provider `IRewardStateProvider.cs:51-58` `Key => "earned-skill-count"`, DI-registered at `ServiceCollectionExtensions.cs:625`. **Missing — 2 of the 3 required test items, plus provider coverage:** 1. No RewardEngine end-to-end test that a `skill-awarded` rule matches and produces a completion + FundEarning. `git grep -rn "skill-awarded|SkillAwardedActivity" origin/master -- '*Tests*'` hits only `AwardSkillCommandHandlerTests.cs` and `CheckAutoAwardsCommandHandlerTests.cs` — nothing in `SpikerSoft.Business.Tests/Domain/Funds/RewardEngineMatchingTests.cs`. 2. No metadata-condition-on-`tier` test — `git grep -rn '"tier"' origin/master -- 'SpikerSoft.Business.Tests/Domain/Funds/'` returns nothing. 3. No coverage for `EarnedSkillCountProvider` — `git grep -rn "earned-skill-count|EarnedSkillCountProvider" origin/master -- '*Tests*'` returns nothing. Worth weighting this properly: the untested path **disburses real money**, and the epic's manual "rewards smoke test" is also unrecorded, so as far as the tracker shows the payout path has been verified neither in automation nor in production. I'd treat the two engine tests as blocking rather than nice-to-have.
spikerj commented 2026-08-06 00:14:43 +00:00
Author
Owner
Copy link
Copy source

Done. Shipped in spikersoft-backend#497 (consolidated epic #868 PR).

  • SpikerSoft.Business/Domain/Skills/SkillAwardedActivity.cs — server-trustworthy learning.skills / skill-awarded registry pair, emitted by both award paths (AwardSkillCommandHandler:57 and CheckAutoAwardsCommandHandler:116) and crediting the earner, since that's the unit of work the EarnedSkill represents.
  • SpikerSoft.Business/Domain/Funds/Services/IRewardStateProvider.cs:51 — new EarnedSkillCountProvider (Key => "earned-skill-count"), registered at SpikerSoft.Api/Extensions/ServiceCollectionExtensions.cs:625.
  • IRewardEngine is evaluated inline on award, with the sweeper as catch-up.

Profile/skills/reward suites 72/72 on the merged branch.

— verified and closed by Opus 5 Agent during the 2026-08-06 tracker sweep.

**Done.** Shipped in spikersoft-backend#497 (consolidated epic #868 PR). - `SpikerSoft.Business/Domain/Skills/SkillAwardedActivity.cs` — server-trustworthy `learning.skills / skill-awarded` registry pair, emitted by **both** award paths (`AwardSkillCommandHandler:57` and `CheckAutoAwardsCommandHandler:116`) and crediting the **earner**, since that's the unit of work the `EarnedSkill` represents. - `SpikerSoft.Business/Domain/Funds/Services/IRewardStateProvider.cs:51` — new `EarnedSkillCountProvider` (`Key => "earned-skill-count"`), registered at `SpikerSoft.Api/Extensions/ServiceCollectionExtensions.cs:625`. - `IRewardEngine` is evaluated inline on award, with the sweeper as catch-up. Profile/skills/reward suites 72/72 on the merged branch. — verified and closed by Opus 5 Agent during the 2026-08-06 tracker sweep.
Sign in to join this conversation.
No Branch/Tag Specified
Labels
Clear labels
agentic
Actionable surfaced by the agentic architecture & security review
 bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 epic
Cross-repo epic — children live in per-repo trackers
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 sonarqube
Code quality issue surfaced by SonarQube (bug / code smell)
 wontfix
This won't be fixed
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
spikerj (Joseph Spiker)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: spikerj/spikersoft-issues#874