0xrsydn/idx-cli
1
mirror of https://github.com/0xrsydn/idx-cli.git synced 2026-11-07 01:35:36 +00:00
Code Issues Projects Releases 1 Packages Wiki Activity

fix(actions): take the old kind names as input too

Browse source 
JSON `kind` now uses the `--kind` names, so an agent that copies a kind from
an answer cached or read before that change (`public_expose`,
`tender_offer`) would get INVALIDINPUT. Both are accepted, as hidden `--kind`
aliases and by the MCP tools.

Claude-Session: https://claude.ai/code/session_01RqPPVeVDrnLoP7NUZmLRxY
This commit is contained in:
Rasyidan Akbar F. 2026-11-04 19:23:48 +07:00
commit a0bf478ec2
1 changed files with 21 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

23
crates/idx-core/src/corporate/types.rs
View file

@ -28,11 +28,17 @@ pub enum ActionKind {
/// Shareholders' meeting (RUPS). /// Shareholders' meeting (RUPS).
Meeting, Meeting,
/// Public expose. /// Public expose.
#[cfg_attr(feature = "clap", value(name = "expose"))] #[cfg_attr(
feature = "clap",
value(name = "expose", alias = "public-expose", alias = "public_expose")
)]
#[serde(rename = "expose", alias = "public_expose")] #[serde(rename = "expose", alias = "public_expose")]
PublicExpose, PublicExpose,
Ipo, Ipo,
#[cfg_attr(feature = "clap", value(name = "tender"))] #[cfg_attr(
feature = "clap",
value(name = "tender", alias = "tender-offer", alias = "tender_offer")
)]
#[serde(rename = "tender", alias = "tender_offer")] #[serde(rename = "tender", alias = "tender_offer")]
TenderOffer, TenderOffer,
} }
@ -103,6 +109,13 @@ impl std::str::FromStr for ActionKind {
fn from_str(value: &str) -> Result<Self, Self::Err> { fn from_str(value: &str) -> Result<Self, Self::Err> {
let wanted = value.trim().to_ascii_lowercase().replace('_', "-"); let wanted = value.trim().to_ascii_lowercase().replace('_', "-");
// The names JSON carried before they matched `--kind`, which an
// agent may pass back from an earlier answer.
let wanted = match wanted.as_str() {
"public-expose" => "expose",
"tender-offer" => "tender",
other => other,
};
Self::ALL Self::ALL
.into_iter() .into_iter()
.find(|kind| kind.as_str() == wanted) .find(|kind| kind.as_str() == wanted)
@ -333,5 +346,11 @@ mod tests {
// Entries cached before the names changed still read. // Entries cached before the names changed still read.
let old: ActionKind = serde_json::from_str("\"public_expose\"").unwrap(); let old: ActionKind = serde_json::from_str("\"public_expose\"").unwrap();
assert_eq!(old, ActionKind::PublicExpose); assert_eq!(old, ActionKind::PublicExpose);
// And as input, so a kind copied from an older answer works.
assert_eq!("public_expose".parse::<ActionKind>().unwrap(), old);
assert_eq!(
"tender_offer".parse::<ActionKind>().unwrap(),
ActionKind::TenderOffer
);
} }
} }