Skip to content

feat(pushin): add raw file url - #239

Merged
danielroe merged 2 commits into
danielroe:mainfrom
WilcoSp:pushin/rawurl
Oct 24, 2026
Merged

danielroe merged 2 commits into
danielroe:mainfrom
WilcoSp:pushin/rawurl

Conversation

@WilcoSp

@WilcoSp WilcoSp commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

🔗 Linked issue

None so far I know

🧭 Context

This pr adds the raw file url for Pushin.
The raw file url is documented here on pushin's docs

🧪 Testing

I've added to "builds raw file and commit URLs on %s" the raw file url that should be expected

📚 Docs

I haven't changed the docs as there is no difference compared to other forges.
If docs changes are needed please let me know.

Play the walkthrough

@coldtea-pr-lens

coldtea-pr-lens Bot commented Oct 23, 2026 •

Copy link
Copy Markdown
  • Redraw · drawn at 0da26f8, the branch has moved on

Architecture

Architecture diagram for danielroe/forges at 0da26f8

Play the walkthrough


Data flow

No data-flow sequence changed in this PR.


Tip

Set github.comment.collapsed: true in .github/pr-lens.yml to fold the comment behind one View architecture and data flow row. Drawing still runs as before

🪧 More tips
  • Run npx skills add coldteadotai/pr-lens, then tell your coding agent: "Diagram the change you just made with PR Lens and attach it to the pull request."
  • Run npx @coldtea/pr-lens-cli analyze --base origin/main on a branch, then npx @coldtea/pr-lens-cli render .pr-lens/graph.json. Same lenses, your own model key, before the pull request exists
  • Click the link under each diagram to open it on a canvas you can zoom, pan and step through
  • The diagrams are links. Click one to open it on the canvas, then press W or click play to walk through the change
  • Open a diagram on the canvas, then press W or click play to walk through the change one step at a time
  • The CLI's render reads .github/pr-lens.yml and applies your renames, exclusions and lane pins at draw time
  • Set github.draw: on-demand in .github/pr-lens.yml and PR Lens stops drawing pull requests on its own. Comment @pr-lens draw on a pull request when you want that one drawn
  • Add .github/workflows/pr-lens.yml with coldteadotai/pr-lens/packages/action@v0 and your model provider's key as its api-key to run PR Lens from your own CI. Any /chat/completions endpoint works
  • After a push, tick Redraw to draw the latest commit
  • Switch GitHub to dark mode and the diagrams follow. The moving dots are this pull request's data in motion
  • Set github.description: false in .github/pr-lens.yml to stop PR Lens adding buttons to your PR descriptions

Thanks for using PR Lens! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out on X, Bluesky, Mastodon, Reddit or LinkedIn

@vercel

vercel Bot commented Oct 23, 2026

Copy link
Copy Markdown

@WilcoSp is attempting to deploy a commit to the danielroe Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 34 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 2 included reviews currently available.

Learn how review limits work.

Review configuration:

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: a4f232a7-80b8-4c6f-bc94-e44d04a910fe
Commits

Reviewing files that changed from the base of the PR and between 0da26f8 and fac82de.

Files selected for processing: 2
  • src/pushin/web.ts
  • test/unit/identity.test.ts

No actionable comments were generated in the recent review.

ℹ️ Recent review info
⚙️ Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: bf8884fb-07ea-4dce-8d45-1081906d5da1
📥 Commits

Reviewing files that changed from the base of the PR and between 40d50b2 and 0da26f8.

📒 Files selected for processing: 2
  • src/pushin/web.ts
  • test/unit/identity.test.ts

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.

Walkthrough

The Pushin link configuration now enables raw links and sets rawOrigin to https://raw.pushin.eu. Three empty async-generator expressions in the identity tests receive formatting-only changes.

Changes

Pushin raw links

Layer / File(s) Summary
Link configuration and test formatting
src/pushin/web.ts, test/unit/identity.test.ts
The Pushin configuration enables raw links and sets their origin. Three empty async-generator expressions in the tests gain a space; test behaviour and expectations do not change.

Priority: Normal

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Feature

Suggested reviewers: danielroe

Merge Risk: Minimal · up to 0da26

No actionable merge-blocking risk is evident in the reviewed changes.

Pre-merge checks: Passed 4 · Failed 1

Failed Failed checks: 1 warning

Check name Status Explanation Resolution
Docstring Coverage Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 2 files. Write docstrings for the functions missing them to satisfy the coverage threshold.
Passed Passed checks: 4 passed
Check name Status Explanation
Description check Passed The description clearly explains that the pull request adds raw file URL support for Pushin and updates the related test expectation.
Title check Passed The title clearly and concisely identifies the main change: adding a raw file URL for Pushin.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches: 💡 1
🛠️ Fix failing CI checks: 💡
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@pkg-pr-new

pkg-pr-new Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/danielroe/forges@239

commit: fac82de

@github-actions

github-actions Bot commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Bundle size

Browser entry bundles compared with the PR base. All sizes are in bytes. Lazy totals the minified chunks besides the entry, which load on demand or never.

Bundle Minified Gzip Lazy
forges 4,070 → 4,065 (-5, -0.1%) 1,800 → 1,793 (-7, -0.4%) 49,179 → 49,285 (+106, +0.2%)
forges/github 138,311 → 139,455 (+1,144, +0.8%) 42,113 → 42,307 (+194, +0.5%) 28,522 → 29,906 (+1,384, +4.9%)
forges/gitlab 99,264 → 100,671 (+1,407, +1.4%) 30,926 → 31,231 (+305, +1.0%) 10,042
forges/tangled 90,553 → 95,411 (+4,858, +5.4%) 29,434 → 31,153 (+1,719, +5.8%) 11,832 → 11,766 (-66, -0.6%)
forges/gitea 96,303 → 97,572 (+1,269, +1.3%) 29,803 → 30,057 (+254, +0.9%) 8,414
forges/bitbucket 75,633 → 75,984 (+351, +0.5%) 24,204 → 24,240 (+36, +0.1%) 3,936
forges/web 7,989 → 8,132 (+143, +1.8%) 3,423 → 3,472 (+49, +1.4%) 0

Size changes are informational. pnpm test:size checks entry chunks for webhook translation, and the provider-free bundle against its budget.

Measured for fac82dec678c2eb20e6298d72041fe73c843e522.

@danielroe
danielroe merged commit cb37311 into danielroe:main Oct 24, 2026
13 of 14 checks passed
@danielroe

Copy link
Copy Markdown
Owner

thank you ❤️

@github-actions github-actions Bot mentioned this pull request Oct 24, 2026
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