Surface contributor starting points on the hub#941
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThis PR adds a "Contributor starting points" feature to the Mergework hub. It introduces a new data constant, extends the hub context function to transform and expose that data, renders a new template section with styled entry cards, and adds both unit and integration tests to verify the context transformation and final HTML output. ChangesContributor starting points feature
Possibly related PRs
🚥 Pre-merge checks | ✅ 6✅ Passed checks (6 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
catcherintheroad-hub
left a comment
There was a problem hiding this comment.
Reviewed current head 69f6d3d40e5ef2f757803fe07dd87e107e798d09 for #838.
This is a narrow public hub-page routing change. I inspected app/hub.py, app/templates/hub.html, and tests/test_hub.py: the new MERGEWORK_CONTRIBUTOR_STARTING_POINTS entries point only to existing public read-only surfaces for effectively open bounties, accepted-work activity, and the open-bounty JSON feed. mergework_hub_context() returns fresh dict copies for the starting-point list, and the template renders those links with normal Jinja escaping.
Validation on this exact head:
uv run --python 3.12 --extra dev python -m pytest tests/test_hub.py -q-> 5 passed, 1 existing Starlette/httpx warning.uv run --python 3.12 --extra dev ruff check app/hub.py tests/test_hub.py-> passed.uv run --python 3.12 --extra dev ruff format --check app/hub.py tests/test_hub.py-> 2 files already formatted.uv run --python 3.12 --extra dev mypy app/hub.py-> success.uv run --python 3.12 --extra dev python scripts/docs_smoke.py-> docs smoke ok.git diff --check origin/main...HEAD-> clean.git merge-tree --write-tree origin/main HEAD-> clean treecb246f3bc4bcce158d2d0a907cd97e342f9932c2.
GitHub state checked before review: PR open, non-draft, mergeStateStatus=CLEAN, hosted Quality/readiness/docs/image check successful, CodeRabbit successful, and no human reviews visible on this head.
Scope stays limited to public hub-page contributor navigation and tests. No bounty API response shape, attempt creation, payout execution, treasury/proposal execution, ledger mutation, wallet behavior, admin-token behavior, private data, credentials, bridge/exchange/cash-out behavior, or MRWK price behavior changed or claimed.
Bounty #845
Summary
Why this fits #845
This is a small contributor-routing improvement on the first public MergeWork page. It points contributors and agents to existing public surfaces that confirm whether a bounty is live, effectively open, pending, or already paid before they start work.
Duplicate / current check
app/hub.py,app/templates/hub.html, ortests/test_hub.py.Validation
uv run --python 3.12 --extra dev python -m pytest tests/test_hub.py -q-> 5 passed, 1 existing Starlette/httpx warning.uv run --python 3.12 --extra dev ruff check app/hub.py tests/test_hub.py-> passed.uv run --python 3.12 --extra dev ruff format --check app/hub.py tests/test_hub.py-> 2 files already formatted.uv run --python 3.12 --extra dev mypy app/hub.py-> success.uv run --python 3.12 --extra dev python scripts/docs_smoke.py-> docs smoke ok.Scope
Public hub-page routing only. No bounty API shape, attempt creation, payout execution, treasury mutation, wallet behavior, ledger mutation, admin-token behavior, private data, exchange, bridge, cash-out, or MRWK price behavior changed.
Summary by CodeRabbit
New Features
Tests