Skip to content

Fix delivery funnel session and PR counts - #245

Merged
Aymen (aymenfurter) merged 1 commit into
mainfrom
aymenfurter-ai-coach-dashboard
Aug 26, 2026
Merged

Fix delivery funnel session and PR counts#245
Aymen (aymenfurter) merged 1 commit into
mainfrom
aymenfurter-ai-coach-dashboard

Conversation

@aymenfurter

@aymenfurter Aymen (aymenfurter) commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • derive the delivery funnel from durable project-session history instead of retained workspace rows
  • count canonical issue references, structured PR references, and strict local merge evidence
  • keep the seven-day workspace PR cohort chart separate from all-time session delivery metrics
  • clarify the UI labels so they describe locally evidenced links and confirmed merges
  • share canonical issue and PR SQL/parsing logic across analytics and issue-credit calculations

Checks

  • npm test — 1,348 tests passed
  • npm run typecheck
  • npm run build
  • npm run spellcheck
  • npm run check-size
  • targeted ESLint for changed files

npm run check reached Knip but could not complete because the local package feed does not provide the locked CSpell version and the machine ran out of disk space while restoring validation-only dependencies.

Count project sessions and durable issue and pull request evidence from the local session store while keeping workspace PR cohorts separate.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: b2c65883-5847-4890-9a81-a03ff73e173b
@github-actions

Copy link
Copy Markdown

⚠️ Deprecation Warning: The deny-licenses option is deprecated for possible removal in the next major release. For more information, see issue 997.

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@aymenfurter
Aymen (aymenfurter) merged commit c2cab82 into main Aug 26, 2026
7 checks passed
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