fmt and clippy updated

Author: dipankar

.github/workflows/ci.yml

@@ -0,0 +1,84 @@
+name: CI
+
+on:
+  push:
+    branches: [main]
+  pull_request:
+    branches: [main]
+
+env:
+  CARGO_TERM_COLOR: always
+  RUST_BACKTRACE: 1
+
+jobs:
+  check:
+    name: Check
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: dtolnay/rust-toolchain@stable
+      - uses: Swatinem/rust-cache@v2
+      - run: cargo check --all-targets
+
+  fmt:
+    name: Format
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: dtolnay/rust-toolchain@stable
+        with:
+          components: rustfmt
+      - run: cargo fmt --all -- --check
+
+  clippy:
+    name: Clippy
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: dtolnay/rust-toolchain@stable
+        with:
+          components: clippy
+      - uses: Swatinem/rust-cache@v2
+      - run: cargo clippy --all-targets -- -D warnings
+
+  test:
+    name: Test (${{ matrix.os }})
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu-latest, macos-latest, windows-latest]
+    steps:
+      - uses: actions/checkout@v4
+      - uses: dtolnay/rust-toolchain@stable
+      - uses: Swatinem/rust-cache@v2
+      - run: cargo test --all
+
+  build:
+    name: Build
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: dtolnay/rust-toolchain@stable
+      - uses: Swatinem/rust-cache@v2
+      - run: cargo build --release
+
+  docs:
+    name: Docs
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: dtolnay/rust-toolchain@stable
+      - uses: Swatinem/rust-cache@v2
+      - run: cargo doc --no-deps --all
+        env:
+          RUSTDOCFLAGS: -D warnings
+
+  msrv:
+    name: MSRV (1.75)
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: dtolnay/rust-toolchain@1.75
+      - uses: Swatinem/rust-cache@v2
+      - run: cargo check --all

README.md

@@ -1,5 +1,11 @@
 # SolScript
 
+[![CI](https://github.com/cryptuon/solscript/actions/workflows/ci.yml/badge.svg)](https://github.com/cryptuon/solscript/actions/workflows/ci.yml)
+[![Crates.io](https://img.shields.io/crates/v/solscript-cli.svg)](https://crates.io/crates/solscript-cli)
+[![Documentation](https://docs.rs/solscript-cli/badge.svg)](https://docs.rs/solscript-cli)
+[![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
+[![Rust Version](https://img.shields.io/badge/rust-1.75%2B-blue.svg)](https://www.rust-lang.org)
+
 **Write Solidity. Deploy to Solana.**
 
 SolScript lets you write smart contracts in familiar Solidity syntax and compile them to native Solana programs. No Rust required. No Anchor boilerplate. Just your contract logic.

crates/solscript-ast/src/lib.rs

@@ -252,7 +252,7 @@ pub struct FnDef {
     pub state_mutability: Vec<StateMutability>,
     pub modifiers: Vec<ModifierInvocation>,
     pub return_params: Vec<ReturnParam>,
-    pub body: Option<Block>,  // None for abstract functions
+    pub body: Option<Block>, // None for abstract functions
     pub span: Span,
 }
 
@@ -340,7 +340,7 @@ pub enum Stmt {
     Revert(RevertStmt),
     Delete(DeleteStmt),
     Selfdestruct(SelfdestructStmt),
-    Placeholder(Span),  // _ in modifiers
+    Placeholder(Span), // _ in modifiers
     Expr(ExprStmt),
 }
 
@@ -504,7 +504,7 @@ pub enum BinaryOp {
     Mul,
     Div,
     Rem,
-    Exp,  // **
+    Exp, // **
     // Comparison
     Eq,
     Ne,
@@ -532,13 +532,13 @@ pub struct UnaryExpr {
 
 #[derive(Debug, Clone, Copy, PartialEq, Eq, Serialize, Deserialize)]
 pub enum UnaryOp {
-    Not,      // !
-    Neg,      // -
-    BitNot,   // ~
-    PreInc,   // ++x
-    PreDec,   // --x
-    PostInc,  // x++
-    PostDec,  // x--
+    Not,     // !
+    Neg,     // -
+    BitNot,  // ~
+    PreInc,  // ++x
+    PreDec,  // --x
+    PostInc, // x++
+    PostDec, // x--
 }
 
 #[derive(Debug, Clone, PartialEq, Serialize, Deserialize)]

crates/solscript-ast/src/types.rs

@@ -75,7 +75,7 @@ impl TypePath {
     }
 
     /// Get the full path as a string
-    pub fn to_string(&self) -> String {
+    pub fn full_path(&self) -> String {
         self.segments
             .iter()
             .map(|s| s.name.as_str())
@@ -106,7 +106,7 @@ pub struct MappingType {
 #[derive(Debug, Clone, PartialEq, Serialize, Deserialize)]
 pub struct ArrayType {
     pub element: TypePath,
-    pub sizes: Vec<Option<u64>>,  // None = dynamic [], Some(n) = fixed [n]
+    pub sizes: Vec<Option<u64>>, // None = dynamic [], Some(n) = fixed [n]
     pub span: Span,
 }
 
@@ -227,7 +227,7 @@ pub enum PrimitiveType {
 
 impl PrimitiveType {
     /// Try to parse a primitive type from a string
-    pub fn from_str(s: &str) -> Option<Self> {
+    pub fn parse(s: &str) -> Option<Self> {
         match s {
             // uint aliases
             "uint" | "uint256" => Some(Self::Uint256),