-
-
Notifications
You must be signed in to change notification settings - Fork 1
feat(DevTools): Modernize gRPC ui with Svelte 5 and Vite #49
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
5122419
d7c2e03
13abcb3
a50ab90
ccc177b
9edd472
0256efa
624c6e3
cadc069
321dbd5
ea7cba0
b447b83
47d77c1
a687319
a68eda8
98f5ad6
11eeb97
6341546
d9c7201
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,4 @@ | ||
| [ADR Home](../../README.md) | [Category Index](./README.md) | [Previous](./025-plugin-options-openapi-and-marten-integrations.md) | [Next]() | ||
| [ADR Home](../../README.md) | [Category Index](./README.md) | [Previous](./025-plugin-options-openapi-and-marten-integrations.md) | [Next](./027-devtools-ui-typescript.md) | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win 🔎 Supported by static analysis🏁 Script executed: #!/bin/bash
set -euo pipefail
test -f Docs/ADR/027-devtools-ui-typescript.mdRepository: AuthKits/AuthKit.Server Length of output: 161 🏁 Script executed: set -euo pipefail
printf '%s\n' '--- tracked ADR files ---'
git ls-files 'Docs/ADR/*' | sort | tail -n 15
printf '%s\n' '--- ADR 026 links ---'
cat -n Docs/ADR/026-plugin-authentication-and-authorization-hooks.md | sed -n '1,55p'
printf '%s\n' '--- ADR index references ---'
rg -n '026-plugin-authentication-and-authorization-hooks|027-devtools-ui-typescript|Next|Previous' Docs/ADR/README.md Docs/ADR 2>/dev/null | head -n 80Repository: AuthKits/AuthKit.Server Length of output: 16794 🤖 get_repo_knowledge executed:
Length of output: 15091 Add the missing ADR or update the navigation links. 🧰 Tools🪛 markdownlint-cli2 (0.23.2)[warning] 1-1: First line in a file should be a top-level heading (MD041, first-line-heading, first-line-h1) 🤖 Prompt for AI Agents |
||
|
|
||
| # [ADR-026] Configure Plugin Authentication And Authorization Through Host Security Infrastructure | ||
|
|
||
|
|
@@ -46,4 +46,4 @@ Plugin schemes and policies can protect plugin endpoints through normal ASP.NET | |
| - [ADR-023](./023-plugin-application-pipeline-hooks.md) - plugin endpoints and middleware pipeline | ||
| - [Issue #12](https://github.com/AuthKits/AuthKit.Server/issues/12) - authentication and authorization hooks | ||
|
|
||
| [ADR Home](../../README.md) | [Category Index](./README.md) | [Previous](./025-plugin-options-openapi-and-marten-integrations.md) | [Next]() | ||
| [ADR Home](../../README.md) | [Category Index](./README.md) | [Previous](./025-plugin-options-openapi-and-marten-integrations.md) | [Next](./027-devtools-ui-typescript.md) | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win
🔎 Supported by static analysis
🏁 Script executed:
Repository: AuthKits/AuthKit.Server
Length of output: 3168
🏁 Script executed:
Repository: AuthKits/AuthKit.Server
Length of output: 10402
🏁 Script executed:
Repository: AuthKits/AuthKit.Server
Length of output: 50379
🏁 Script executed:
Repository: AuthKits/AuthKit.Server
Length of output: 12100
🏁 Script executed:
Repository: AuthKits/AuthKit.Server
Length of output: 20509
Document the actual JSON wire contract.
DevToolsMiddlewareuses ASP.NET Core JSON options, withJsonSerializerDefaults.Webas the fallback. The host and plugin do not override these options. Therefore,ResponseBase64andIsPluginare emitted asresponseBase64andisPlugin, and nullable properties remain present withnullvalues. Update the ADR and add an endpoint contract test for casing and null emission.🤖 Prompt for AI Agents