fix: strip @ prefix if present

Author: danielroe

app/utils/npm.ts

@@ -2,10 +2,10 @@
  * Constructs a scope:team string in the format expected by npm.
  * npm operations require the format @scope:team (with @ prefix).
  *
- * @param orgName - The organization name (without @)
+ * @param orgName - The organization name (with or without @)
  * @param teamName - The team name
  * @returns The scope:team string in @scope:team format
  */
 export function buildScopeTeam(orgName: string, teamName: string): string {
-  return `@${orgName}:${teamName}`
+  return `@${orgName.replace(/^@/, '')}:${teamName}`
 }

test/unit/npm-utils.spec.ts

@@ -9,6 +9,11 @@ describe('buildScopeTeam', () => {
     expect(buildScopeTeam('nuxt', 'core')).toBe('@nuxt:core')
   })
 
+  it('strips existing @ prefix from orgName', () => {
+    expect(buildScopeTeam('@netlify', 'developers')).toBe('@netlify:developers')
+    expect(buildScopeTeam('@nuxt', 'core')).toBe('@nuxt:core')
+  })
+
   it('produces format accepted by validateScopeTeam', () => {
     expect(() => validateScopeTeam(buildScopeTeam('netlify', 'developers'))).not.toThrow()
     expect(() => validateScopeTeam(buildScopeTeam('nuxt', 'core'))).not.toThrow()