Skip to content

Feat/parsing/parser reborn#133

Merged
Mahe-Evan merged 47 commits intomainfrom
feat/parsing/parser-reborn
Jan 18, 2026
Merged

Feat/parsing/parser reborn#133
Mahe-Evan merged 47 commits intomainfrom
feat/parsing/parser-reborn

Conversation

@Fenriir42
Copy link
Copy Markdown
Contributor

@Fenriir42 Fenriir42 commented Jan 17, 2026

Pull Request

Description

Add the parser (lexer + parser) for the Glados, this is a V1 it doens't handle everything from the ast
but all mandatory features and more

Related Story/Task

Closes #129, #130, #131 #132

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing
    functionality to not work as expected)
  • 📚 Documentation update
  • ♻️ Refactor (no functional changes)
  • 🧪 Test addition or modification
  • ⚙️ Build/CI changes

Implementation Details

A begining of a repl was added

Checklist

  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
    (including regression and/or integration tests)
  • Any dependent changes have been merged and published
  • The related story/task is updated

Fenriir42 and others added 30 commits January 16, 2026 13:18
@Fenriir42 @Ciznia
parser: implement lexer for parsing tokens
1397e12
@Fenriir42 @Ciznia
parser: add Token data type for lexical units
e579f67
@Fenriir42 @Ciznia
parser: add custom error type for lexical errors
90f9310
@Fenriir42 @Ciznia
parser: implement lexer functions for string and file input
2cf810f
@Fenriir42 @Ciznia
parser: update parser.cabal to include new exposed modules and deps
fff9ba9
@Ciznia
parser: factorize some function
be2c66e
@Fenriir42 @Ciznia
parser: refactor error handling and improve lexer functionality
a5090e3
@Sigmanificient @Ciznia
various: fix version number
a2b8287
@Mahe-Evan @Ciznia
lisp: add tests rules
3de608e
@Fenriir42 @Ciznia
parser: implement lexer and parser executable, add comprehensive tests
e912e5b
@Ciznia
parser: remove outdated code
b0f6375
@Ciznia
parser: add localisation to token
4dfafb6
@Ciznia
parser: add Token -> AssignOp
fd93e0c 
The rest of the token to ast conversion should be a lot quicker now that I have a starter fundation
@Fenriir42 @Ciznia
repl: add REPL module with command parsing and interaction
41254ed
@Fenriir42 @Ciznia
parser: fix warnings and errors
c7fc1a5
@Ciznia
parser: add language extension
1ec3862
@Fenriir42 @Ciznia
parser: change string to text
06f2591
@Fenriir42 @Ciznia
parser: refactor non-decimal number handling
f19a370
@Ciznia
parser: make custom show for token, for intuitive hex representation
14a0ef4
@Ciznia
parser: implement operator to ast and almost all literals
b563df8
@Fenriir42 @Ciznia
parser: enhance literal parsing with array and float support
9c635b1
@Fenriir42 @Ciznia
parser: add primitive type parsing
d8e9355
@Ciznia
parser: fix test according to latest changes
9e6f414
@Ciznia
parser: handle negative number properly
ead892d
@Ciznia
parser: add support for all typing except dropped feature
3fdc5cf
@Fenriir42 @Ciznia
parser: add expression parsing
ff2d984
@Ciznia
parser: token -> AST of a lot of statement, and function declaration
de434f3
@Ciznia
parser: use the defined voidSpan consitently
76c6464
@Ciznia
parser: split in smaller files
7e13bdb
@Ciznia
ast: add + as a unary operator
6f9699f
@github-project-automation github-project-automation Bot moved this to Created in GlaDOS Jan 17, 2026
@renman-ymd renman-ymd added the 5. status: waiting-for-review This PR is waiting for reviewer feedback label Jan 17, 2026
@github-actions github-actions Bot added 7. component: parser Code related to the parser that processes source code syntax. 7. component: nix Code related to Nix configuration, flakes, or the Nix build system. 7. component: tests Code related to the test suite, test framework, test utilities, or test coverage. 7. component: ast Code related to the abstract syntax tree representation of source code 7. component: dep-manager Code related with dependency management and package resolution and removed 5. status: waiting-for-review This PR is waiting for reviewer feedback labels Jan 17, 2026
@renman-ymd renman-ymd moved this from Created to In Review in GlaDOS Jan 17, 2026
@github-actions github-actions Bot added 1. difficulty : hard Complex task requiring significant expertise or deep research. 8. part: 2-custom-language Second deliverable: Custom language with compiler and VM. labels Jan 17, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 17, 2026

⚠️ Release label required

This PR targets main but has no release label (6. release: major/minor/patch).

Please add one to trigger automatic version bumping:

  • major: Breaking changes (x.0.0)
  • minor: New features (0.x.0)
  • patch: Bug fixes (0.0.x)

@Fenriir42 Fenriir42 added 6. release: major Breaking changes requiring major version bump (x.0.0). 1. difficulty : medium Requires some investigation or moderate technical effort. 2. priority: critical Immediate action needed; blocks progress or causes major risk. 3. type: feature Adds new functionality or capability to the project. labels Jan 17, 2026
@Fenriir42 Fenriir42 mentioned this pull request Jan 17, 2026
Merged
@Fenriir42 Fenriir42 force-pushed the feat/parsing/parser-reborn branch from 1e06f6f to fea9499 Compare January 17, 2026 23:18
@renman-ymd renman-ymd added the 5. status: waiting-for-review This PR is waiting for reviewer feedback label Jan 17, 2026
@Ciznia Ciznia removed the 1. difficulty : medium Requires some investigation or moderate technical effort. label Jan 18, 2026
@Ciznia Ciznia force-pushed the feat/parsing/parser-reborn branch from 1e17118 to fb5224c Compare January 18, 2026 12:01
@renman-ymd renman-ymd added 5. status: ready-for-merge This PR is approved and ready to be merged and removed 5. status: waiting-for-review This PR is waiting for reviewer feedback labels Jan 18, 2026
@renman-ymd renman-ymd moved this from In Review to Waiting Merge in GlaDOS Jan 18, 2026
@Mahe-Evan Mahe-Evan merged commit 7fcb2a9 into main Jan 18, 2026
12 checks passed
@github-project-automation github-project-automation Bot moved this from Waiting Merge to Closed in GlaDOS Jan 18, 2026
@Ciznia Ciznia deleted the feat/parsing/parser-reborn branch January 18, 2026 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Mahe-Evan Mahe-Evan Mahe-Evan approved these changes

@renman-ymd renman-ymd Awaiting requested review from renman-ymd

Assignees

@Fenriir42 Fenriir42

@Ciznia Ciznia

Labels

1. difficulty : hard Complex task requiring significant expertise or deep research. 2. priority: critical Immediate action needed; blocks progress or causes major risk. 3. type: feature Adds new functionality or capability to the project. 5. status: ready-for-merge This PR is approved and ready to be merged 6. release: major Breaking changes requiring major version bump (x.0.0). 7. component: ast Code related to the abstract syntax tree representation of source code 7. component: dep-manager Code related with dependency management and package resolution 7. component: nix Code related to Nix configuration, flakes, or the Nix build system. 7. component: parser Code related to the parser that processes source code syntax. 7. component: tests Code related to the test suite, test framework, test utilities, or test coverage. 8. part: 2-custom-language Second deliverable: Custom language with compiler and VM.

Projects

GlaDOS
Status: Closed

Milestone

2. Defense - Finale

Development

Successfully merging this pull request may close these issues.

[EPIC] Parser

5 participants

@Fenriir42 @Mahe-Evan @Ciznia @renman-ymd @Sigmanificient