feat: improve Danger testing and conventional commit scope handling

- Add comprehensive testing infrastructure with 23 test cases
- Fix scope handling for conventional commits (feat(core): -> feat)
- Properly classify ref commits as internal changes
- Add modular architecture with testable functions
- Include CI integration for JavaScript testing

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: vaind

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 ### Features
 
+- Danger - Improve testing infrastructure and conventional commit scope handling ([#105](https://github.com/getsentry/github-workflows/pull/105))
 - Add Proguard artifact endpoint for Android builds in sentry-server ([#100](https://github.com/getsentry/github-workflows/pull/100))
 
 ### Security

danger/dangerfile-utils.js

@@ -7,11 +7,7 @@ const FLAVOR_CONFIG = [
   },
   {
     labels: ["fix", "bug", "bugfix"],
-    changelog: "Bug Fixes"  // More standard than "Fixes"
-  },
-  {
-    labels: ["ref"],  // Very common in Sentry repos (14 occurrences)
-    changelog: "Changes"
+    changelog: "Fixes"
   },
   {
     labels: ["sec", "security"],
@@ -22,17 +18,35 @@ const FLAVOR_CONFIG = [
     changelog: "Performance"
   },
   {
-    labels: ["docs", "doc", "style", "refactor", "tests", "test", "build", "ci", "chore", "meta", "deps", "dep", "chore(deps)", "build(deps)"],
-    changelog: undefined  // Internal changes - no changelog needed
+    // Internal changes - no changelog needed
+    changelog: undefined,
+    labels: [
+      "docs",
+      "doc",
+      "style",
+      "ref",
+      "refactor",
+      "tests",
+      "test",
+      "build",
+      "ci",
+      "chore",
+      "meta",
+      "deps",
+      "dep"
+    ]
   }
 ];
 
 /// Get flavor configuration for a given PR flavor
 function getFlavorConfig(prFlavor) {
   const normalizedFlavor = prFlavor.toLowerCase().trim();
 
+  // Strip scope/context from conventional commit format: "type(scope)" -> "type"
+  const baseType = normalizedFlavor.replace(/\([^)]*\)/, '');
+
   const config = FLAVOR_CONFIG.find(config =>
-    config.labels.includes(normalizedFlavor)
+    config.labels.includes(normalizedFlavor) || config.labels.includes(baseType)
   );
 
   return config || {

danger/dangerfile-utils.test.js

@@ -16,15 +16,15 @@ describe('dangerfile-utils', () => {
 
     it('should return config for fixes without isFeature', () => {
       const fixConfig = getFlavorConfig('fix');
-      assert.strictEqual(fixConfig.changelog, 'Bug Fixes');
+      assert.strictEqual(fixConfig.changelog, 'Fixes');
       assert.strictEqual(fixConfig.isFeature, undefined);
 
       const bugConfig = getFlavorConfig('bug');
-      assert.strictEqual(bugConfig.changelog, 'Bug Fixes');
+      assert.strictEqual(bugConfig.changelog, 'Fixes');
       assert.strictEqual(bugConfig.isFeature, undefined);
 
       const bugfixConfig = getFlavorConfig('bugfix');
-      assert.strictEqual(bugfixConfig.changelog, 'Bug Fixes');
+      assert.strictEqual(bugfixConfig.changelog, 'Fixes');
       assert.strictEqual(bugfixConfig.isFeature, undefined);
     });
 
@@ -53,7 +53,7 @@ describe('dangerfile-utils', () => {
       assert.strictEqual(config1.changelog, 'Features');
 
       const config2 = getFlavorConfig(' fix ');
-      assert.strictEqual(config2.changelog, 'Bug Fixes');
+      assert.strictEqual(config2.changelog, 'Fixes');
     });
 
     it('should handle all security-related flavors', () => {
@@ -72,11 +72,24 @@ describe('dangerfile-utils', () => {
       assert.strictEqual(performanceConfig.changelog, 'Performance');
     });
 
-    it('should handle ref flavor (common in Sentry repos)', () => {
+    it('should handle ref flavor (internal changes - no changelog)', () => {
       const refConfig = getFlavorConfig('ref');
-      assert.strictEqual(refConfig.changelog, 'Changes');
+      assert.strictEqual(refConfig.changelog, undefined);
       assert.strictEqual(refConfig.isFeature, undefined);
     });
+
+    it('should handle scoped flavors by stripping scope', () => {
+      const scopedFeat = getFlavorConfig('feat(core)');
+      assert.strictEqual(scopedFeat.changelog, 'Features');
+      assert.strictEqual(scopedFeat.isFeature, true);
+
+      const scopedFix = getFlavorConfig('fix(browser)');
+      assert.strictEqual(scopedFix.changelog, 'Fixes');
+      assert.strictEqual(scopedFix.isFeature, undefined);
+
+      const scopedChore = getFlavorConfig('chore(deps)');
+      assert.strictEqual(scopedChore.changelog, undefined);
+    });
   });
 
   describe('extractPRFlavor', () => {