Address PR review feedback and fix CI issues

- Remove duplicate method definitions in parse_v3.jl that caused precompilation failures
- Remove unsupported operators from documentation (** power operator, % modulo operator)
- Fix while loop example in README to not use unimplemented assignments
- Fix Aqua persistent tasks test (now passes after removing duplicate methods)

All tests now pass successfully including Aqua quality checks.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: GiggleLiu

README.md

@@ -95,7 +95,7 @@ This package currently supports the following OpenQASM 3.0 features:
 - **Control Flow**: `if-else`, `while`, `for` loops, `break`, `continue`
   ```julia
   if (c == 1) { x q; } else { h q; }
-  while (i < 10) { i = i + 1; }
+  while (c == 0) { x q; }
   for int i in [0:10] { ... }
   for int i in {1, 5, 10} { ... }
   ```
@@ -141,7 +141,7 @@ Important differences to be aware of when migrating from QASM 2.0 to 3.0:
 
 2. **New Syntax**: Prefer `qubit[n]` over `qreg`, and `bit[n]` over `creg` in QASM 3.0 (though legacy syntax is still supported)
 
-3. **Enhanced Expressions**: QASM 3.0 supports logical operators (`&&`, `||`), comparison operators (`==`, `!=`, `<`, `>`, `<=`, `>=`), and power operator (`**`)
+3. **Enhanced Expressions**: QASM 3.0 supports logical operators (`&&`, `||`) and comparison operators (`==`, `!=`, `<`, `>`, `<=`, `>=`)
 
 ## Roadmap
 

src/parse_v3.jl

@@ -14,10 +14,6 @@ struct QASM3Lang end
 # Customize struct names to avoid collisions with QASM 2.0
 RBNF.typename(::Type{QASM3Lang}, name::Symbol) = Symbol(:QASM3_, name)
 
-# QASM 3.0 specific type conversions (only define what's unique to v3.0)
-Base.convert(::Type{Bool}, t::Token{:reserved}) = (t.str == "const")
-Base.convert(::Type{Bool}, ::Nothing) = false
-
 # RBNF crate methods for QASM 3.0 specific types
 RBNF.crate(::Type{TypesV3.QASMType}) = TypesV3.IntType()
 RBNF.crate(::Type{TypesV3.IntType}) = TypesV3.IntType()
@@ -181,7 +177,7 @@ RBNF.@parser QASM3Lang begin
     # ========== Expressions ==========
 
     # Expression grammar with operator precedence
-    # Supports: logical (&&, ||), comparison (==, !=, <, >, <=, >=), arithmetic (+, -, *, /, %), power (**)
+    # Supports: logical (&&, ||), comparison (==, !=, <, >, <=, >=), arithmetic (+, -, *, /)
 
     expr = logical_or_expr
 

test/aqua.jl

@@ -39,8 +39,6 @@ using OpenQASM
 
     # Test for persistent tasks
     @testset "Persistent tasks" begin
-        # Skip persistent tasks test - it fails due to precompilation issues
-        # with RBNF.Token wrapper constructor
-        Aqua.test_persistent_tasks(OpenQASM; broken=true)
+        Aqua.test_persistent_tasks(OpenQASM)
     end
 end