Skip to content

fix(deps): switch from deprecated serde_yaml to yaml_serde fork #81 - #121

Merged
araujof merged 1 commit into
praxis-proxy:mainfrom
aviavissar:fix--switch-from-deprecated-serde_yaml-to-yaml_serde-fork--#81
Sep 24, 2026
Merged

araujof merged 1 commit into
praxis-proxy:mainfrom
aviavissar:fix--switch-from-deprecated-serde_yaml-to-yaml_serde-fork--#81

Conversation

@aviavissar

@aviavissar aviavissar commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Issue

praxis-proxy/policy#81 — Replace the deprecated serde_yaml crate with the maintained yaml_serde fork (yaml.org).

Closes #81

What was done

A single line was changed in praxis-policy/Cargo.toml (workspace root):

Before:

serde_yaml = "0.9"

After:

# NOTE: serde_yaml is deprecated; using yaml_serde (yaml.org maintained fork) as drop-in.
serde_yaml = { package = "yaml_serde", version = "0.10.4" }

This uses Cargo's package rename feature: the actual crate downloaded from crates.io is yaml_serde, but all source code continues to reference serde_yaml:: unchanged. The 10 member crates inherit the dependency via { workspace = true }, so no other Cargo.toml files needed editing. Zero .rs source files were modified.

Cargo.lock was regenerated by cargo update, pulling in yaml_serde v0.10.7 and its dependency libyaml-rs v0.3.0.

Files changed

  • praxis-policy/Cargo.toml — 1 line changed (dependency declaration)
  • praxis-policy/Cargo.lock — regenerated automatically

What was verified

Step Command Result
Dependency resolution cargo update yaml_serde v0.10.7 + libyaml-rs v0.3.0 resolved successfully
Compilation cargo build --workspace All crates compiled without errors
Tests cargo test --workspace 84 test suites passed, 0 failures
Linting cargo clippy --workspace --all-targets 0 warnings

Prior art

The sibling repo (praxis-proxy/proxy) already completed the same migration using the identical technique (praxis/Cargo.toml line 75-76).

@aviavissar
aviavissar requested a review from a team September 22, 2026 09:56
@araujof araujof added the chore Infra, dependency updates, etc. label Sep 23, 2026
@araujof araujof moved this from Backlog to In progress in Praxis Policy Engine (PPE) Sep 23, 2026
@araujof araujof added the rust Pull requests that update rust code label Sep 23, 2026
@araujof araujof added this to the 0.4.0 milestone Sep 23, 2026
@aviavissar
aviavissar force-pushed the fix--switch-from-deprecated-serde_yaml-to-yaml_serde-fork--#81 branch 2 times, most recently from eee2c96 to 34af692 Compare September 24, 2026 11:31

@araujof araujof left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

…s-proxy#81

Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: aviavissar <aviavissar@gmail.com>
@araujof
araujof force-pushed the fix--switch-from-deprecated-serde_yaml-to-yaml_serde-fork--#81 branch from 34af692 to 07ed228 Compare September 24, 2026 15:38
@araujof
araujof merged commit 7fba4bc into praxis-proxy:main Sep 24, 2026
7 checks passed
araujof pushed a commit that referenced this pull request Sep 24, 2026
Signed-off-by: aviavissar <aviavissar@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Infra, dependency updates, etc. rust Pull requests that update rust code

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

fix: switch from deprecated serde_yaml to yaml_serde fork

3 participants