Skip to content

Feat: unified key-value store abstraction and adapters #165

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.

Sign up for GitHub

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

Open
prk-Jr wants to merge 18 commits into
base: main
Choose a base branch
from
+3,838 −52
Open

Feat: unified key-value store abstraction and adapters #165

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
ff0fec4
Implements a provider-neutral KV store interface with support for Axu…
Feb 15, 2026
b1aff0c
Fixes formatting issues detected by CI check, primarily in app-demo h…
Feb 15, 2026
fc69ee2
Fix clippy lints
Feb 15, 2026
5ac528c
feat: Enforce KV key, value, and TTL limits with new validation logic…
Feb 16, 2026
857b2cd
test: Directly instantiate MemoryKvStore in tests to bypass minimum T…
Feb 16, 2026
a133876
Axum adapter now uses a persistent redb database for local KV storage…
Feb 17, 2026
0fb3340
fix lint error
Feb 17, 2026
41d3398
Merge branch 'main' into feat/kv-store
aram356 Feb 21, 2026
dd1b831
Rename modules to and update related documentation and dependencies…
Feb 23, 2026
371b515
Use SystemTime instead of Instant in MockStore for WASM compatibility.
Feb 24, 2026
83e832f
Merge branch 'main' into feat/kv-store
Feb 27, 2026
e36dcaa
Update KV action return types to , refine KV error handling, and add …
Feb 27, 2026
676d0c7
Improve Cloudflare adapter HTTP method handling, update key-value st…
Feb 27, 2026
f968890
Rename KV store to , add bounded body draining, and simplify Cloudfl…
Mar 6, 2026
760da54
Remove list_keys api
Mar 10, 2026
586699d
Impl list_keys_page
Mar 11, 2026
463861f
Merge branch 'main' into feat/kv-store
aram356 Mar 11, 2026
c7b8b7b
harden kv startup semantics and verify wasm adapter targets
Mar 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 8 additions & 2 deletions .github/workflows/test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,8 +43,8 @@ jobs:
with:
toolchain: ${{ steps.rust-version.outputs.rust-version }}

- name: Add wasm32-wasi target
run: rustup target add wasm32-wasip1
- name: Add wasm targets
run: rustup target add wasm32-wasip1 wasm32-unknown-unknown

- name: Setup Viceroy
run: cargo install viceroy --locked
Expand All @@ -57,3 +57,9 @@ jobs:

- name: Check feature compilation
run: cargo check --workspace --all-targets --features "fastly cloudflare"

- name: Check Fastly wasm target
run: cargo check -p edgezero-adapter-fastly --features fastly --target wasm32-wasip1

- name: Check Cloudflare wasm target
run: cargo check -p edgezero-adapter-cloudflare --features cloudflare --target wasm32-unknown-unknown
4 changes: 4 additions & 0 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,8 +1,12 @@
# node
node_modules/

# compiled output
bin/
pkg/
target/
.wrangler/
.edgezero/

# env
.env
Expand Down
14 changes: 14 additions & 0 deletions Cargo.lock
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions Cargo.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,7 @@ log = "0.4"
log-fastly = "0.11"
matchit = "0.9"
once_cell = "1"
redb = "3.1.0"
reqwest = { version = "0.13", default-features = false, features = ["rustls"] }
serde = { version = "1", features = ["derive"] }
serde_json = "1"
Expand Down
23 changes: 20 additions & 3 deletions crates/edgezero-adapter-axum/Cargo.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,26 @@ license = { workspace = true }

[features]
default = ["axum"]
axum = ["dep:axum", "dep:tokio", "dep:tower", "dep:futures-util", "dep:reqwest"]
cli = ["dep:edgezero-adapter", "edgezero-adapter/cli", "dep:ctor", "dep:toml", "dep:walkdir"]
axum = [
"dep:axum",
"dep:tokio",
"dep:tower",
"dep:futures-util",
"dep:reqwest",
"dep:redb",
]
Copy link
Contributor

@aram356 aram356 Mar 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Low: serde_json is added to the axum feature gate but never used in adapter source code.

grep -r serde_json crates/edgezero-adapter-axum/src/ returns zero matches. Same for Fastly adapter (Cargo.toml line 37). This adds unnecessary build graph churn.

If these were added speculatively for future use, remove them and add back when needed.

cli = [
"dep:edgezero-adapter",
"edgezero-adapter/cli",
"dep:ctor",
"dep:toml",
"dep:walkdir",
]

[dependencies]
edgezero-adapter = { path = "../edgezero-adapter", optional = true, features = ["cli"] }
edgezero-adapter = { path = "../edgezero-adapter", optional = true, features = [
"cli",
] }
edgezero-core = { path = "../edgezero-core" }
anyhow = { workspace = true }
async-trait = { workspace = true }
Expand All @@ -22,6 +37,7 @@ futures = { workspace = true }
futures-util = { workspace = true, optional = true }
http = { workspace = true }
log = { workspace = true }
redb = { workspace = true, optional = true }
reqwest = { workspace = true, optional = true }
simple_logger = { workspace = true }
thiserror = { workspace = true }
Expand All @@ -34,5 +50,6 @@ walkdir = { workspace = true, optional = true }
[dev-dependencies]
async-trait = { workspace = true }
axum = { workspace = true, features = ["macros"] }
serde = { workspace = true }
tempfile = { workspace = true }
tokio = { workspace = true, features = ["macros", "rt", "rt-multi-thread"] }
Loading
Loading