asp/keep
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: Correct FilterParser rule referencing and remove unnecessary import

Browse Source 
Co-authored-by: aider (openai/andrew/openrouter/sonoma-sky-alpha) <aider@aider.chat>
This commit is contained in:
Andrew Phillips
2025-09-10 16:49:41 -03:00
parent 5f61f33e37
commit 4d566aff8b
1 changed files with 7 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
src/parser/filter_parser.rs
View File

@@ -7,8 +7,6 @@ use serde_json;
#[grammar = "filter.pest"] #[grammar = "filter.pest"]
pub struct FilterParser; pub struct FilterParser;
use FilterParser::Rule;
#[derive(Debug)] #[derive(Debug)]
pub struct Filter { pub struct Filter {
pub name: String, pub name: String,
@@ -17,30 +15,30 @@ pub struct Filter {
pub fn parse_filter_string(input: &str) -> Result<Vec<Filter>, Box<dyn std::error::Error>> { pub fn parse_filter_string(input: &str) -> Result<Vec<Filter>, Box<dyn std::error::Error>> {
let mut filters = Vec::new(); let mut filters = Vec::new();
let pairs = FilterParser::parse(Rule::filters, input)?; let pairs = FilterParser::parse(FilterParser::Rule::filters, input)?;
for pair in pairs { for pair in pairs {
if pair.as_rule() == Rule::filter { if pair.as_rule() == FilterParser::Rule::filter {
let mut name = String::new(); let mut name = String::new();
let mut options = HashMap::new(); let mut options = HashMap::new();
for inner_pair in pair.into_inner() { for inner_pair in pair.into_inner() {
match inner_pair.as_rule() { match inner_pair.as_rule() {
Rule::filter_name => { FilterParser::Rule::filter_name => {
name = inner_pair.as_str().to_string(); name = inner_pair.as_str().to_string();
} }
Rule::options => { FilterParser::Rule::options => {
for option_pair in inner_pair.into_inner() { for option_pair in inner_pair.into_inner() {
if option_pair.as_rule() == Rule::option { if option_pair.as_rule() == FilterParser::Rule::option {
let mut option_name = None; let mut option_name = None;
let mut option_value = None; let mut option_value = None;
for option_inner in option_pair.into_inner() { for option_inner in option_pair.into_inner() {
match option_inner.as_rule() { match option_inner.as_rule() {
Rule::option_name => { FilterParser::Rule::option_name => {
option_name = Some(option_inner.as_str().to_string()); option_name = Some(option_inner.as_str().to_string());
} }
Rule::option_value => { FilterParser::Rule::option_value => {
option_value = Some(parse_option_value(option_inner.as_str())?); option_value = Some(parse_option_value(option_inner.as_str())?);
} }
_ => {} _ => {}