Merge from upstream #39
Open
yumin-chen wants to merge 155 commits into
Open
Conversation
1. When `git archive` errors, do not require an exit code, but just use
the `ExitStatus` we already have. This requires less unwrapping.
Also, the previous default of 0 was misleading, since 0 is returned
upon successful termination.
2. Use "refname" instead of "refspec" as these are different concepts.
Refer to <https://git-scm.com/docs/git-check-ref-format> and in
particular
<https://git-scm.com/docs/git-check-ref-format#Documentation/git-check-ref-format.txt---refspec-pattern>.
3. Only allow appending ".tar.gz" to commit IDs, not to refnames,
because refnames may contain the suffix ".tar.gz". It might look
weird, but it is not forbidden to do `git branch foo.tar.gz`.
4. `char::is_ascii` does not help sanitize refnames to file names. In
particular we have `'/'.is_ascii() == true`. Replace forward slashes
with two underscores.
To allow some very basic hyperlinking in commit descriptions we can replace fully specified urls with a valid scheme into `radicle-external-link` components.
Based on the work done in commit `a73c5490b160435a0ef5408b54185c9f698cb73f` there are a few places that make the app not load in case dom storage aka localStorage is disabled.
The `-c` argument (and its corresponding value) must come *before* the first subcommand to be effective. This error was introduced in commit 5f22e1b.
Instead of hardcoding the version of the Rust alpine file, introduce an `ARG` in the `Dockerfile`. In `build/build`, we determine the version by inspecting `rust-toolchain.toml`, and pass this value in. This ensures that the binaries are built using the same toolchain that we develop with. Co-authored-by: Fintan Halpenny <fintan.halpenny@gmail.com>
Don't run prettier on a rust crate
This reverts commit 9e9a799.
It should always be generated from the Node Id.
To allow some very basic hyperlinking in commit descriptions we can replace fully specified urls with a valid scheme into `radicle-external-link` components within patches and git history.
Replace the hand-rolled URL regex with `linkify-it` (already a transitive dep via `marked-linkify-it`). Add unit tests.
Replace global setConfig with an exported sanitizeConfig passed inline. The global config leaked into mermaid's internal strict-mode sanitization, stripping valid SVG diagrams.
'loose' allowed HTML and click directives in diagram labels, bypassing the DOMPurify pass on the surrounding markdown.
Switch to a dynamic import in afterUpdate and drop manualChunks so Rolldown auto-splits the lazy chunk. Eager bundle drops from 1.3 MB to 610 KB gzipped.
Hoist the dynamic import and initialize() into a module-scoped promise. Previous code re-ran initialize() on every afterUpdate that had a mermaid block.
When mermaid.render fails, suppress mermaid's "Syntax error" overlay and show our own small warning above the original code block so the source stays visible. Subsequent diagrams in the same document still render.
This prevents flakes in the visual test suite due to stack frame naming
mismatches in the command copy code box.
E.g.
```diff
-{"stack":"Error: Response error\n at Dx.fet
+{"stack":"Error: Response error\n at wx.fet
```
Bump radicle-surf to 0.27.1 Bump radicle-term to 0.18.0 Bump radicle-crypto to 0.17.0
These crates are interlocked: radicle-job 0.6.0 requires radicle 0.24.x, so they must be bumped together.
Public seeds return `"agent": null` because their deployed httpd binaries can't parse i2p/onion rows that newer nodes write to the address table — `AddressStore::get` fails and the handler falls back to `None`. Allow null in the zod schema and hide the user-agent row in the UI when it's missing. Workaround until heartwood ships `StoreExt::agent_of` (reads only the `agent` column, no address hydration) and httpd adopts it.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
13 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.