Skip to content

fix(release): map carrier contributor emails - #44290

Closed
claw3649 wants to merge 1 commit into
NousResearch:mainfrom
claw3649:fix/release-author-map
Closed

fix(release): map carrier contributor emails#44290
claw3649 wants to merge 1 commit into
NousResearch:mainfrom
claw3649:fix/release-author-map

Conversation

@claw3649

Copy link
Copy Markdown

Summary

  • add missing AUTHOR_MAP aliases for the contributor emails currently tripping the carrier attribution check
  • map the unnumbered konsisumer@users.noreply.github.com noreply form alongside the existing numbered noreply entry
  • map the maplestoryjuni222@gmail.com commit email to the already-known BROCCOLO1D account
  • map claw3649@gmail.com to claw3649

Evidence

  • carrier attribution check reported these unmapped emails: claw3649@gmail.com, konsisumer@users.noreply.github.com, maplestoryjuni222@gmail.com
  • konsisumer@users.noreply.github.com is the standard unnumbered noreply form for konsisumer; the numbered form is already in AUTHOR_MAP
  • maplestoryjuni222@gmail.com is verified by GitHub commit API for 2903615: author.login is BROCCOLO1D and commit.author.email is maplestoryjuni222@gmail.com

Validation

git diff --check -- scripts/release.py
/Users/claw2/.hermes/hermes-agent/.venv/bin/python -m py_compile scripts/release.py
/Users/claw2/.hermes/hermes-agent/.venv/bin/python - <<'PY'
import importlib.util
from pathlib import Path
repo = Path('/tmp/hermes-release-author-map')
spec = importlib.util.spec_from_file_location('release', repo / 'scripts' / 'release.py')
m = importlib.util.module_from_spec(spec)
spec.loader.exec_module(m)
expected = {
    'claw3649@gmail.com': 'claw3649',
    'konsisumer@users.noreply.github.com': 'konsisumer',
    'maplestoryjuni222@gmail.com': 'BROCCOLO1D',
}
for email, user in expected.items():
    assert m.AUTHOR_MAP.get(email) == user
    assert m.resolve_author(user, email) == f'@{user}'
print('AUTHOR_MAP mappings OK')
PY

Add missing AUTHOR_MAP aliases for carrier commits so the contributor attribution check can resolve the current CI carrier authors.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@alt-glitch alt-glitch added type/refactor Code restructuring, no behavior change P3 Low — cosmetic, nice to have comp/cli CLI entry point, hermes_cli/, setup wizard labels Jun 11, 2026
@claw3649 claw3649 closed this Jun 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard P3 Low — cosmetic, nice to have type/refactor Code restructuring, no behavior change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants