refactor(core/assembler): rm interface and export class directly

Author: exuanbo

src/core/assembler/assembler.spec.ts

@@ -2,12 +2,12 @@ import { describe, expect, it } from 'vitest'
 
 import { examples } from '@/features/editor/examples'
 
-import { createAssembler } from './assembler'
+import { Assembler } from './assembler'
 
 describe('Assembler', () => {
   examples.forEach(({ title, content }) => {
     it(`should assemble example ${title}`, () => {
-      const assembler = createAssembler()
+      const assembler = new Assembler()
       const unit = assembler.run(content)
       expect(unit.ast).not.toBeNull()
       expect(unit.chunks.length).toBeGreaterThan(0)
@@ -17,7 +17,7 @@ describe('Assembler', () => {
   })
 
   it('should collect parser errors', () => {
-    const assembler = createAssembler()
+    const assembler = new Assembler()
     const unit = assembler.run('inc al')
     expect(unit.errors).toHaveLength(1)
     expect(unit.errors[0]).toMatchInlineSnapshot(
@@ -26,7 +26,7 @@ describe('Assembler', () => {
   })
 
   it('should collect parser warnings', () => {
-    const assembler = createAssembler()
+    const assembler = new Assembler()
     const unit = assembler.run('label: end')
     expect(unit.warnings).toHaveLength(1)
     expect(unit.warnings[0]).toMatchInlineSnapshot(
@@ -35,7 +35,7 @@ describe('Assembler', () => {
   })
 
   it('should validate jump distances', () => {
-    const assembler = createAssembler()
+    const assembler = new Assembler()
     const unit = assembler.run('jmp label org 81 label: end')
     expect(unit.errors).toHaveLength(1)
     expect(unit.errors[0]).toMatchInlineSnapshot(
@@ -44,7 +44,7 @@ describe('Assembler', () => {
   })
 
   it('should validate immediate values', () => {
-    const assembler = createAssembler()
+    const assembler = new Assembler()
     const unit = assembler.run('add al, 100 end')
     expect(unit.errors).toHaveLength(1)
     expect(unit.errors[0]).toMatchInlineSnapshot(
@@ -53,7 +53,7 @@ describe('Assembler', () => {
   })
 
   it('should validate string literals', () => {
-    const assembler = createAssembler()
+    const assembler = new Assembler()
     const unit = assembler.run('db "你好世界" end')
     expect(unit.errors).toHaveLength(1)
     expect(unit.errors[0]).toMatchInlineSnapshot(
@@ -62,7 +62,7 @@ describe('Assembler', () => {
   })
 
   it('should validate org address', () => {
-    const assembler = createAssembler()
+    const assembler = new Assembler()
     const unit = assembler.run('org 100 end')
     expect(unit.errors).toHaveLength(1)
     expect(unit.errors[0]).toMatchInlineSnapshot(
@@ -71,7 +71,7 @@ describe('Assembler', () => {
   })
 
   it('should throw an error when memory overflows', () => {
-    const assembler = createAssembler()
+    const assembler = new Assembler()
     const unit = assembler.run('org ff inc al end')
     expect(unit.errors).toHaveLength(1)
     expect(unit.errors[0]).toMatchInlineSnapshot(
@@ -80,7 +80,7 @@ describe('Assembler', () => {
   })
 
   it('should merge errors', () => {
-    const assembler = createAssembler()
+    const assembler = new Assembler()
     const unit = assembler.run('org ff inc al inc bl inc cl end')
     expect(unit.errors).toHaveLength(1)
     expect(unit.errors[0]).toMatchInlineSnapshot(

src/core/assembler/assembler.ts

@@ -21,14 +21,6 @@ import { createParser } from './parser'
 import { createParserContext, ParserContext, useParserContext } from './parser.context'
 import { createTokenStream, TokenStream } from './token.stream'
 
-export interface Assembler {
-  run(input: string): AssemblyUnit
-}
-
-export function createAssembler(): Assembler {
-  return new AssemblerImpl()
-}
-
 interface PendingChunk extends CodeChunk {
   node: WithIdentifier<ProcessableNode>
 }
@@ -40,7 +32,7 @@ type ResolvableNode =
     ? Exclude<Values, AST.Mnemonic>
     : never
 
-class AssemblerImpl implements Assembler {
+export class Assembler {
   private pendings!: PendingChunk[]
   private unit!: AssemblyUnit