Consolidate open AI bot PRs: Bolt optimizations and dependabot update#60
Draft
Consolidate open AI bot PRs: Bolt optimizations and dependabot update#60
Conversation
…ytes dependency Merges changes from the following open AI bot PRs: - PRs #34-43: Clippy fixes, field shorthand, lifetime annotations, iterator optimizations - PRs #45-55: Value Display, parser list/map/chain expr optimizations - PRs #56-57: Token string allocation optimizations (already in codebase) - PR #33: Bump bytes 1.4.0 -> 1.11.1 (dependabot) - PR #38: Value Display formatting (already in codebase) Changes applied: - parser.rs: Remove unnecessary & on op args, into_iter -> iter in describe(), push_str("x") -> push('x'), borrow instead of clone in map_expr, lifetime annotations - tokenizer.rs: Field shorthand, remove unnecessary return, use is_ascii_*() methods, lifetime annotations, Copy instead of clone for Token - operator.rs: Field shorthand for all managers, *precedence instead of clone() - function.rs: Field shorthand - descriptor.rs: Use ? operator for early return - lib.rs: Explicit lifetime annotation on parse_expression - .jules/bolt.md: Add Display optimization learning - Cargo.lock: Bump bytes 1.4.0 -> 1.11.1 Agent-Logs-Url: https://github.com/ashyanSpada/expression_engine_rs/sessions/e76f8236-8653-4bc9-bf47-86b983ff48e3 Co-authored-by: ashyanSpada <22587148+ashyanSpada@users.noreply.github.com>
Agent-Logs-Url: https://github.com/ashyanSpada/expression_engine_rs/sessions/e76f8236-8653-4bc9-bf47-86b983ff48e3 Co-authored-by: ashyanSpada <22587148+ashyanSpada@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Merge merge requests created by AI bot
Consolidate open AI bot PRs: Bolt optimizations and dependabot update
Apr 8, 2026
ashyanSpada
approved these changes
Apr 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Merges remaining changes from ~25 open Bolt AI PRs (#34–#57) and dependabot PR #33 into a single commit. Many PRs had overlapping or already-merged changes (Value Display, Context lock optimizations, Token
as_str()); this extracts only the delta.Clippy / idiomatic Rust fixes (from PR #34)
&onopargs inexec_unary/exec_binary,into_iter→iterindescribe(), lifetime annotations oncur_tok/nextreturn,is_ascii_digit()/is_ascii_lowercase()/is_ascii_uppercase()instead of manual range checks, lifetime annotations,Copyinstead of.clone()forToken{ store }not{ store: store })*precedenceinstead ofprecedence.clone()fori32is_none()+unwrap()with.cloned()Parser expr formatting (from PR #55)
list_expr/map_expr/chain_expr:push('x')instead ofpush_str("x"), borrow instead of clone inmap_expr, drop&")".to_string()inbinary_exprDependency update (from PR #33)
bytes1.4.0 → 1.11.1Bolt learnings
.jules/bolt.md