Tv

bug: Using environment variables for conditional compilation breaks use as a Rust library

2

### Did you check existing issues? - [x] I have read all the [tree-sitter docs](https://tree-sitter.github.io/tree-sitter/using-parsers) if it relates to using the parser - [x] I have searched the existing issues...

`#![deny(missing_docs)]` fails on derived archived enum variants

2

This looks like #217 again. It seems ArchivedFoo enum variants don't have doc strings. Either they could get docstrings, or `#[expect(missing_docs, reason = "generated code")]` or something. ```rust //! Repro...

BTreeMap requires Archived keys

2

Hi. I'm trying to use `BTreeMap` and.. is it just me or does it seem like ArchivedBTreeMap is really hard to use? It's `get` method demands the *archived* key type,...

`Portable` trait does not consider indirect types

3

I was chasing an error from miri until I stumbled on https://github.com/rkyv/rkyv/issues/436#issuecomment-1767450952 and with that flag it still panicked, but differently. Take out miri and it still panics. Huh. ```...

While making a reproducer for #596, I stumbled on this. Adding rkyv::Serialize to a derive breaks any `attr` used on enum variants. ```rust #[derive(rkyv::Archive)] pub enum Works { #[rkyv(attr(expect(missing_docs)))] Variant...

examples/tool-call-stream: Duplicates tool arguments

1

I think this: https://github.com/64bit/async-openai/blob/b26346d3efaeb9861259cd0f2e9c8b8f87793148/examples/tool-call-stream/src/main.rs#L82-L86 and this: https://github.com/64bit/async-openai/blob/b26346d3efaeb9861259cd0f2e9c8b8f87793148/examples/tool-call-stream/src/main.rs#L90-L96 store the arguments to a tool call twice. The later call to parse https://github.com/64bit/async-openai/blob/b26346d3efaeb9861259cd0f2e9c8b8f87793148/examples/tool-call-stream/src/main.rs#L225 fails: ``` thread 'tokio-runtime-worker' panicked at src/bin/tool-call-stream.rs:243:57: called `Result::unwrap()`...

Snowflake LIST @foo and REMOVE @foo stage syntax fail to parse with a semicolon

1

```sql -- breaks LIST @bar; -- ok LIST @bar -- breaks REMOVE @bar; -- ok REMOVE @bar ``` This is starting to become a theme. See #1244, #1519. 1. can...

https://docs.rs/sqlparser/0.55.0/sqlparser/ast/enum.ColumnOption.html#variant.ForeignKey says > A referential integrity constraint (`[FOREIGN KEY REFERENCES () { [ON DELETE ] [ON UPDATE ] | [ON UPDATE ] [ON DELETE ] } []`). I believe the...

https://docs.rs/sqlparser/0.55.0/sqlparser/ast/enum.ColumnOption.html#variant.Materialized sqlparser-rs docs say > `MATERIALIZE Syntax: b INT MATERIALIZE (a + 1)` That's missing a D. Both the enum variant name and ClickHouse docs spell it `MATERIALIZED`. https://clickhouse.com/docs/sql-reference/statements/create/table#default_values >...

Make `CreateTable.location` / `AlterTable.location` make more sense

1

`CreateTable.location` has no documentation and is a tricky edge case, not always set even when SQL statement did set location. https://docs.rs/sqlparser/latest/sqlparser/ast/struct.CreateTable.html#structfield.location Since this took some time to figure out, here's...