jspurefix 2.2.1 → 3.0.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jspurefix",
-  "version": "2.2.1",
+  "version": "3.0.0",
   "description": "pure node js fix engine",
   "keywords": [
     "typescript",
@@ -24,57 +24,58 @@
   },
   "scripts": {
     "circular": "madge --circular --extensions js dist && madge --circular --extensions ts src",
-    "test": "jest",
+    "test": "jest --maxWorkers=1 --coverage",
     "build": "tsc",
     "test:debug": "node --inspect-brk node_modules/.bin/jest --runInBand",
-    "cmd": "cd dist && node jsfix-cmd",
-    "repo44": "cd dist && node jsfix-cmd --dict=repo44",
-    "repo44-unit": "cd dist && node jsfix-cmd --dict=repo44 --generate --density=0.8 --unit --delimiter=\"|\" --session=data/session/test-initiator.json",
-    "repo44-script": "cd dist && node jsfix-cmd --dict=repo44 --generate --density=0.8 --script --delimiter=\"|\" --session=data/session/test-initiator.json",
-    "repo44-replay": "cd dist && node jsfix-cmd --dict=repo44 --fix=data/examples/FIX.4.4/fix.txt --delimiter=\"|\" --session=data/session/test-initiator.json",
-    "repo44-repscr": "cd dist && node jsfix-cmd --dict=repo44 --fix=dist/fix.txt --delimiter=\"|\" --stats",
-    "repo44-bench": "cd dist && node jsfix-cmd --dict=repo44 --benchmark --delimiter=\"|\"",
-    "repo44-bench-er": "cd dist && node jsfix-cmd --dict=repo44 --fix=data/examples/FIX.4.4/repo/execution-report/fix.txt --benchmark --delimiter=\"|\" --repeats=125000",
-    "repo44-bench-tc": "cd dist && node jsfix-cmd ---dict=repo44 --fix=data/examples/FIX.4.4/repo/trade-capture/fix.txt --benchmark --delimiter=\"|\" --repeats=30000",
-    "repo44-bench-sd": "cd dist && node jsfix-cmd --dict=repo44 --fix=data/examples/FIX.4.4/repo/security-definition/fix.txt --benchmark --delimiter=\"|\" --repeats=150000",
-    "repo44-bench-lo": "cd dist && node jsfix-cmd --dict=repo44 --fix=data/examples/FIX.4.4/repo/logon/fix.txt --benchmark --delimiter=\"|\" --repeats=250000",
-    "qf-bench-lo": "cd dist && node jsfix-cmd --session=data/session/test-initiator.json --fix=data/examples/FIX.4.4/quickfix/logon/fix.txt --benchmark --delimiter=\"|\" --repeats=250000",
-    "fixml": "cd dist && node jsfix-cmd --dict=repofixml",
-    "tcp-qf-md": "cd dist/sample/tcp/qf-md && node app",
-    "tcp-tls-tc": "cd dist/sample/tcp/tls-trade-capture && node app",
-    "tcp-tc": "cd dist/sample/tcp/trade-capture && node app",
-    "tcp-rec-sk": "cd dist/sample/tcp/recovering-skeleton && node app",
-    "tcp-sk": "cd dist/sample/tcp/skeleton && node app",
-    "http-oms": "cd dist/sample/http/oms && node app",
-    "repo40-compile": "cd dist && node jsfix-cmd \"--dict=repo40\" \"--compile\"",
-    "repo41-compile": "cd dist && node jsfix-cmd \"--dict=repo41\" \"--compile\"",
-    "repo42-compile": "cd dist && node jsfix-cmd \"--dict=repo42\" \"--compile\"",
-    "repo43-compile": "cd dist && node jsfix-cmd \"--dict=repo43\" \"--compile\"",
-    "repo44-compile": "cd dist && node jsfix-cmd \"--dict=repo44\" \"--compile\"",
-    "repo50-compile": "cd dist && node jsfix-cmd \"--dict=repo50\" \"--compile\"",
-    "repo50sp1-compile": "cd dist && node jsfix-cmd \"--dict=repo50sp1\" \"--compile\"",
-    "repo50sp2-compile": "cd dist && node jsfix-cmd \"--dict=repo50sp2\" \"--compile\"",
-    "repofixml-compile": "cd dist && node jsfix-cmd \"--dict=repofixml\" \"--compile\"",
-    "qf44-compile": "cd dist && node jsfix-cmd \"--dict=qf44\" \"--compile\"",
-    "qf43-compile": "cd dist && node jsfix-cmd \"--dict=qf43\" \"--compile\"",
-    "qf42-compile": "cd dist && node jsfix-cmd \"--dict=qf42\" \"--compile\"",
+    "cmd": "node dist/jsfix-cmd",
+    "repo44": "node dist/jsfix-cmd --dict=repo44",
+    "repo44-unit": "node dist/jsfix-cmd --dict=repo44 --generate --density=0.8 --unit --delimiter=\"|\" --session=data/session/test-initiator.json",
+    "repo44-script": "node dist/jsfix-cmd --dict=repo44 --generate --density=0.8 --script --delimiter=\"|\" --session=data/session/test-initiator.json",
+    "repo44-replay": "node dist/jsfix-cmd --dict=repo44 --fix=data/examples/FIX.4.4/fix.txt --delimiter=\"|\" --stats--delimiter=\"|\" --objects",
+    "repo44-repscr": "node dist/jsfix-cmd --dict=repo44 --fix=data/examples/FIX.4.4/fix.txt --delimiter=\"|\" --stats--delimiter=\"|\" --stats",
+    "repo44-bench": "node dist/jsfix-cmd --dict=repo44 --benchmark --delimiter=\"|\"",
+    "qf44-bench-er": "node dist/jsfix-cmd  --dict=qf44 --fix=data/examples/FIX.4.4/quickfix/execution-report/fix.txt --benchmark --delimiter=\"|\" --repeats=80000",
+    "repo44-bench-er": "node dist/jsfix-cmd --dict=repo44 --fix=data/examples/FIX.4.4/repo/execution-report/fix.txt --benchmark --delimiter=\"|\" --repeats=125000",
+    "repo44-bench-tc": "node dist/jsfix-cmd jsfix-cmd ---dict=repo44 --fix=data/examples/FIX.4.4/repo/trade-capture/fix.txt --benchmark --delimiter=\"|\" --repeats=30000",
+    "repo44-bench-sd": "node dist/jsfix-cmd --dict=repo44 --fix=data/examples/FIX.4.4/repo/security-definition/fix.txt --benchmark --delimiter=\"|\" --repeats=150000",
+    "repo44-bench-lo": "node dist/jsfix-cmd --dict=repo44 --fix=data/examples/FIX.4.4/repo/logon/fix.txt --benchmark --delimiter=\"|\" --repeats=250000",
+    "qf-bench-lo": "node dist/jsfix-cmd --session=data/session/test-initiator.json --fix=data/examples/FIX.4.4/quickfix/logon/fix.txt --benchmark --delimiter=\"|\" --repeats=250000",
+    "qf-bench-hb": "node dist/jsfix-cmd --session=data/session/test-initiator.json --fix=data/examples/FIX.4.4/quickfix/heartbeat/fix.txt --benchmark --delimiter=\"|\" --repeats=250000",
+    "fixml": "node dist/jsfix-cmd --dict=repofixml",
+    "tcp-qf-md": "node dist/sample/tcp/qf-md/app",
+    "tcp-tls-tc": "node dist/sample/tcp/tls-trade-capture/app",
+    "tcp-tc": "node dist/sample/tcp/trade-capture/app",
+    "tcp-rec-sk": "node dist/sample/tcp/recovering-skeleton/app",
+    "tcp-sk": "node dist/sample/tcp/skeleton/app",
+    "http-oms": "node dist/sample/http/oms/app",
+    "repo40-compile": "node dist/jsfix-cmd \"--dict=repo40\" \"--compile\"",
+    "repo41-compile": "node dist/jsfix-cmd \"--dict=repo41\" \"--compile\"",
+    "repo42-compile": "node dist/jsfix-cmd \"--dict=repo42\" \"--compile\"",
+    "repo43-compile": "node dist/jsfix-cmd \"--dict=repo43\" \"--compile\"",
+    "repo44-compile": "node dist/jsfix-cmd \"--dict=repo44\" \"--compile\"",
+    "repo50-compile": "node dist/jsfix-cmd \"--dict=repo50\" \"--compile\"",
+    "repo50sp1-compile": "node dist/jsfix-cmd \"--dict=repo50sp1\" \"--compile\"",
+    "repo50sp2-compile": "node dist/jsfix-cmd \"--dict=repo50sp2\" \"--compile\"",
+    "repofixml-compile": "node dist/jsfix-cmd \"--dict=repofixml\" \"--compile\"",
+    "qf44-compile": "node dist/jsfix-cmd \"--dict=qf44\" \"--compile\"",
+    "qf43-compile": "node dist/jsfix-cmd \"--dict=qf43\" \"--compile\"",
+    "qf42-compile": "node dist/jsfix-cmd \"--dict=qf42\" \"--compile\"",
     "unzip-repo": "cd data && node ../src/util/unzip fix_repo.zip",
-    "fixml-test-print": "cd dist && node jsfix-cmd --session=data/session/test-http-initiator.json --fix=data/examples/fixml/jsfix.test_http-client.txt --objects",
-    "fix-test-print": "cd dist && node jsfix-cmd --dict=repo44 --fix=data/examples/FIX.4.4/jsfix.test_client.txt --objects --delimiter=\"|\""
+    "fixml-test-print": "node dist/jsfix-cmd --session=data/session/test-http-initiator.json --fix=data/examples/fixml/jsfix.test_http-client.txt --objects",
+    "fix-test-print": "node dist/jsfix-cmd --dict=repo44 --fix=data/examples/FIX.4.4/jsfix.test_client.txt --objects --delimiter=\"|\""
   },
   "author": "",
   "license": "MIT",
   "dependencies": {
     "align-text": "^1.0.2",
+    "axios": "^1.2.2",
     "express": "^4.18.2",
     "lodash": "^4.17.21",
-    "mathjs": "^11.3.0",
+    "mathjs": "^11.5.0",
     "minimist": "^1.2.7",
     "moment": "^2.29.4",
     "node-fs-extra": "^0.8.2",
     "reflect-metadata": "^0.1.13",
-    "request": "^2.88.2",
-    "request-promise-native": "^1.0.9",
     "sax": "^1.2.4",
     "tsyringe": "^4.7.0",
     "uuid": "^9.0.0",
@@ -83,23 +84,27 @@
     "yauzl": "^2.10.0"
   },
   "devDependencies": {
-    "@types/express": "^4.17.14",
-    "@types/express-serve-static-core": "^4.17.31",
-    "@types/jest": "^29.1.2",
-    "@types/lodash": "^4.14.186",
+    "@types/express": "^4.17.15",
+    "@types/express-serve-static-core": "^4.17.32",
+    "@types/jest": "^29.2.5",
+    "@types/lodash": "^4.14.191",
     "@types/mathjs": "^9.4.1",
     "@types/minimist": "^1.2.2",
-    "@types/node": "^18.11.0",
+    "@types/node": "^18.11.18",
     "@types/request-promise-native": "^1.0.18",
     "@types/sax": "^1.2.4",
-    "@types/uuid": "^8.3.4",
+    "@types/uuid": "^9.0.0",
     "@types/winston": "^2.4.4",
-    "jest": "^29.2.0",
+    "@typescript-eslint/eslint-plugin-tslint": "^5.47.1",
+    "eslint": "^8.31.0",
+    "eslint-config-standard-with-typescript": "^24.0.0",
+    "eslint-plugin-jquery": "^1.5.1",
+    "eslint-plugin-jsdoc": "^39.6.4",
+    "eslint-plugin-node": "^11.1.0",
+    "jest": "^29.3.1",
     "madge": "^5.0.1",
     "standard": "^17.0.0",
     "ts-jest": "^29.0.3",
-    "tslint": "^6.1.3",
-    "tslint-config-standard": "^9.0.0",
-    "typescript": "^4.8.4"
+    "typescript": "^4.9.4"
   }
 }

package/src/buffer/ascii/ascii-chars.ts

@@ -1,5 +1,4 @@
 export abstract class AsciiChars {
-
   public static readonly Minus: number = '-'.charCodeAt(0)
   public static readonly Equal: number = '='.charCodeAt(0)
   public static readonly ForwardSlash: number = '/'.charCodeAt(0)

package/src/buffer/ascii/ascii-encoder.ts

@@ -14,17 +14,16 @@ import { Tags } from '../tag/tags'
 import { TagType } from '../tag/tag-type'
 
 export class AsciiEncoder extends MsgEncoder {
-
   public bodyLengthPos: number
   public msgTypePos: number
   public tags: Tags
   public checkGroups: boolean = true
 
   constructor (public readonly buffer: ElasticBuffer,
-               public readonly definitions: FixDefinitions,
-               public readonly timeFormatter: ITimeFormatter = new TimeFormatter(buffer),
-               public readonly delimiter: number = AsciiChars.Soh,
-               public readonly logDelimiter: number = AsciiChars.Pipe) {
+    public readonly definitions: FixDefinitions,
+    public readonly timeFormatter: ITimeFormatter = new TimeFormatter(buffer),
+    public readonly delimiter: number = AsciiChars.Soh,
+    public readonly logDelimiter: number = AsciiChars.Pipe) {
     super(definitions)
     this.tags = new Tags(definitions)
   }
@@ -87,7 +86,7 @@ export class AsciiEncoder extends MsgEncoder {
     const keys: string[] = Object.keys(o)
     let j: number = 0
     const fields: ContainedField[] = keys.reduce((a: ContainedField[], current: string) => {
-      const field: ContainedField = set.localNameToField.get(current)
+      const field: ContainedField | null = set.localNameToField.get(current)
       if (field) {
         a[j++] = field
       }
@@ -98,7 +97,8 @@ export class AsciiEncoder extends MsgEncoder {
   }
 
   private encodeInstances (o: ILooseObject, gf: ContainedGroupField): void {
-    const noOfField: SimpleFieldDefinition = gf.definition.noOfField
+    const noOfField: SimpleFieldDefinition | null = gf.definition.noOfField
+    if (!noOfField) return
     const instances: ILooseObject[] = o[gf.name] || o[noOfField.name]
 
     const buffer = this.buffer
@@ -167,7 +167,6 @@ export class AsciiEncoder extends MsgEncoder {
     }
 
     switch (tagType) {
-
       case TagType.RawData: {
         // may need to first write raw message length (see below)
         break
@@ -262,7 +261,7 @@ class GroupValidator {
     public readonly test: AsciiEncodeSetSummary = new AsciiEncodeSetSummary()) {
   }
 
-  getSummary (field: number) {
+  getSummary (field: number): AsciiEncodeSetSummary {
     return field === 0 ? this.first : this.test
   }
 
@@ -270,31 +269,34 @@ class GroupValidator {
     const first = this.first
     const test = this.test
     if (field === 0 && first.empty()) {
-      throw new Error(`first group instance has no delimeter present ${this.gf.name}`)
+      throw new Error(`first group instance has no delimiter present ${this.gf.name}`)
     }
     if (field > 0 && test.empty()) {
-      throw new Error(`group instance [${field}] has no delimeter present ${this.gf.name}`)
+      throw new Error(`group instance [${field}] has no delimiter present ${this.gf.name}`)
     }
     if (field > 0) {
-      const firstTag = first.firstSimple.definition.tag
-      const tag = test.firstSimple.definition.tag
+      const firstTag = first.firstSimple?.definition.tag
+      const tag = test.firstSimple?.definition.tag
       if (firstTag !== tag) {
-        throw new Error(`group instance [${field}] inconsisent delimeter ${tag} expected tag ${firstTag}`)
+        const msg = `group instance [${field}] inconsistent delimiter ${tag} expected tag ${firstTag}`
+        throw new Error(msg)
       }
     }
   }
 }
 
 class AsciiEncodeSetSummary {
-  constructor (public firstSimple: ContainedSimpleField = null,
-    public lastSimple: ContainedSimpleField = null,
+  constructor (public firstSimple: ContainedSimpleField | null = null,
+    public lastSimple: ContainedSimpleField | null = null,
     public count: number = 0) {
   }
+
   public reset (): void {
     this.firstSimple = null
     this.lastSimple = null
     this.count = 0
   }
+
   public empty (): boolean {
     return this.firstSimple === null || this.count === 0
   }

package/src/buffer/ascii/ascii-parser-state.ts

@@ -7,7 +7,7 @@ import { DITokens } from '../../runtime/di-tokens'
 
 @injectable()
 export class AsciiParserState {
-  public message: MessageDefinition
+  public message: MessageDefinition | null
   public locations: Tags
   public parseState: ParseState
   public bodyLen: number
@@ -19,12 +19,12 @@ export class AsciiParserState {
   public currentTag: number
   public rawDataLen: number
   public rawDataRead: number
-  public msgType: string
+  public msgType: string | null
 
   constructor (@inject(DITokens.ParseBuffer) public readonly elasticBuffer: ElasticBuffer) {
   }
 
-  public beginTag (pos: number) {
+  public beginTag (pos: number): void {
     this.parseState = ParseState.ParsingTag
     this.tagStartPos = pos
     this.equalPos = this.valueEndPos = -1
@@ -54,7 +54,7 @@ export class AsciiParserState {
 
   public checkRawTag (): void {
     const msg = this.message
-    if (!msg || !msg.containsRaw) {
+    if (!msg?.containsRaw) {
       // optimisation as will never hit raw data
       this.parseState = ParseState.ParsingValue
       return

package/src/buffer/ascii/ascii-parser.ts

@@ -28,12 +28,12 @@ export class AsciiParser extends MsgParser {
   // want to keep one slice of memory and constantly reuse it
 
   constructor (@inject(DITokens.IJsFixConfig) public readonly config: IJsFixConfig,
-               @inject(DITokens.readStream) public readonly readStream: Readable,
-               @inject(DITokens.ParseBuffer) protected readonly receivingBuffer: ElasticBuffer) {
+    @inject(DITokens.readStream) public readonly readStream: Readable | null,
+    @inject(DITokens.ParseBuffer) protected readonly receivingBuffer: ElasticBuffer) {
     super()
 
-    this.delimiter = config.delimiter
-    this.writeDelimiter = config.logDelimiter || AsciiChars.Pipe
+    this.delimiter = config.delimiter ?? AsciiChars.Soh
+    this.writeDelimiter = config.logDelimiter ?? AsciiChars.Pipe
     const definitions = config.definitions
     this.id = AsciiParser.nextId++
     this.segmentParser = config.sessionContainer.resolve<AsciiSegmentParser>(AsciiSegmentParser)
@@ -65,13 +65,13 @@ export class AsciiParser extends MsgParser {
     })
 
     // receive from say a socket or file and pipe to parser which discovers messages
-    stream.pipe(receiver).on('finish', () => {
+    stream?.pipe(receiver).on('finish', () => {
       this.emit('done')
     })
-    stream.on('error', (e) => {
+    stream?.on('error', (e) => {
       this.emit('error', e)
     })
-    stream.on('end', () => {
+    stream?.on('end', () => {
       this.emit('end')
     })
   }
@@ -86,7 +86,7 @@ export class AsciiParser extends MsgParser {
     state.beginMessage()
   }
 
-  public parseText (text: string) {
+  public parseText (text: string): void {
     const buff = Buffer.from(text)
     this.parse(buff, buff.length)
   }
@@ -103,11 +103,10 @@ export class AsciiParser extends MsgParser {
     let readPtr: number = 0
 
     while (readPtr < end) {
-      let charAtPos: number = readBuffer[readPtr]
+      const charAtPos: number = readBuffer[readPtr]
 
       const writePtr = receivingBuffer.saveChar(charAtPos) - 1
       switch (state.parseState) {
-
         case ParseState.MsgComplete: {
           this.msg(writePtr)
           continue
@@ -157,7 +156,8 @@ export class AsciiParser extends MsgParser {
         }
 
         default: {
-          throw new Error(`fix parser in unknown state ${state}`)
+          const st = state.parseState
+          throw new Error(`fix parser in unknown state ${st}`)
         }
       }
       readPtr++
@@ -171,15 +171,18 @@ export class AsciiParser extends MsgParser {
     }
   }
 
-  private getView (ptr: number): MsgView {
+  private getView (ptr: number): MsgView | null {
     const state = this.state
     const locations = state.locations
     const source = this.receivingBuffer
     const delimiter = this.delimiter
     const replace = this.writeDelimiter
+    const msgType = state.msgType ?? null
+    if (!msgType) return null
     if (state.message) {
-      const structure: Structure = this.segmentParser.parse(state.msgType, locations,
+      const structure: Structure | null = this.segmentParser.parse(msgType, locations,
         locations.nextTagPos - 1)
+      if (!structure) return null
       return new AsciiView(structure.msg(),
         source,
         structure,

package/src/buffer/ascii/ascii-segment-parser.ts

@@ -20,14 +20,13 @@ import { SegmentType } from '../segment/segment-type'
 
 @injectable()
 export class AsciiSegmentParser {
-
   constructor (@inject(DITokens.Definitions) public readonly definitions: FixDefinitions) {
   }
 
-  public parse (msgType: string, tags: Tags, last: number): Structure {
+  public parse (msgType: string, tags: Tags, last: number): Structure | null {
     // completed segments in that they are fully parsed
     const segments: SegmentDescription[] = []
-    const msgDefinition: MessageDefinition = this.definitions.message.get(msgType)
+    const msgDefinition: MessageDefinition | null = this.definitions.message.get(msgType)
     if (!msgDefinition) {
       return null
     }
@@ -41,11 +40,12 @@ export class AsciiSegmentParser {
     // having finished one segments keep unwinding until tag matches further up stack
     function unwind (tag: number): void {
       while (structureStack.length > 1) {
-        const done: SegmentDescription = structureStack.pop()
+        const done = structureStack.pop()
+        if (!done) continue
         done.end(segments.length, currentTagPosition - 1, tags.tagPos[currentTagPosition - 1].tag)
         segments.push(done)
         peek = structureStack[structureStack.length - 1]
-        if (peek.set.containedTag[tag]) {
+        if (peek.set?.containedTag[tag]) {
           // unwound to point this tag lives in this set.
           break
         }
@@ -56,10 +56,10 @@ export class AsciiSegmentParser {
       }
     }
 
-    function examine (tag: number): SegmentDescription {
-      let structure: SegmentDescription = null
-      switch (peek.currentField.type) {
-
+    function examine (tag: number): SegmentDescription | null {
+      let structure: SegmentDescription | null = null
+      const type = peek.currentField?.type
+      switch (type) {
         case ContainedFieldType.Simple: {
           const sf: ContainedSimpleField = peek.currentField as ContainedSimpleField
           if (sf.definition.tag === tag) {
@@ -71,14 +71,14 @@ export class AsciiSegmentParser {
         case ContainedFieldType.Component: {
           const cf: ContainedComponentField = peek.currentField as ContainedComponentField
           structure = new SegmentDescription(cf.name, tag, cf.definition,
-                        currentTagPosition, structureStack.length, SegmentType.Component)
+            currentTagPosition, structureStack.length, SegmentType.Component)
           break
         }
         // for a group also need to know where all delimiters are positioned
         case ContainedFieldType.Group: {
           const gf: ContainedComponentField = peek.currentField as ContainedGroupField
           structure = new SegmentDescription(gf.name, tag, gf.definition,
-                        currentTagPosition, structureStack.length, SegmentType.Group)
+            currentTagPosition, structureStack.length, SegmentType.Group)
           currentTagPosition = currentTagPosition + 1
           structure.startGroup(tags.tagPos[currentTagPosition].tag)
           break
@@ -96,8 +96,8 @@ export class AsciiSegmentParser {
       if (tag === peek.delimiterTag) {
         peek.addDelimiterPosition(currentTagPosition)
       } else if (structureStack.length > 1) {
-                // if a group is represented by a repeated component, then the tag representing delimiter
-                // needs to be added further up stack to group itself.
+        // if a group is represented by a repeated component, then the tag representing delimiter
+        // needs to be added further up stack to group itself.
         delimiter = structureStack[structureStack.length - 2].groupAddDelimiter(tag, currentTagPosition)
       }
       return delimiter
@@ -116,7 +116,7 @@ export class AsciiSegmentParser {
         const tag: number = tags.tagPos[currentTagPosition].tag
         peek = structureStack[structureStack.length - 1]
         peek.setCurrentField(tag)
-        if (!peek.set.containedTag[tag] || groupDelimiter(tag)) {
+        if (!peek.set?.containedTag[tag] || groupDelimiter(tag)) {
           // unravelled all way back to root hence this is not recognised
           const unknown = peek.type === SegmentType.Msg
           if (unknown) {
@@ -137,7 +137,8 @@ export class AsciiSegmentParser {
     function clean (): void {
       // any remainder components can be closed.
       while (structureStack.length > 0) {
-        const done: SegmentDescription = structureStack.pop()
+        const done = structureStack.pop()
+        if (!done) continue
         done.end(segments.length, currentTagPosition - 1, tags.tagPos[currentTagPosition - 1].tag)
         segments[segments.length] = done
       }

package/src/buffer/ascii/ascii-view.ts

@@ -12,12 +12,11 @@ import { TagType } from '../tag/tag-type'
 export class AsciiView extends MsgView {
   private readonly timeFormatter: ITimeFormatter = new TimeFormatter(this.buffer)
   constructor (public readonly segment: SegmentDescription,
-               public readonly buffer: ElasticBuffer,
-               public readonly structure: Structure,
-               public readonly ptr: number,
-               public readonly delimiter: number,
-               public readonly writeDelimiter: number) {
-
+    public readonly buffer: ElasticBuffer,
+    public readonly structure: Structure | null,
+    public readonly ptr: number,
+    public readonly delimiter: number,
+    public readonly writeDelimiter: number) {
     super(segment, structure)
   }
 
@@ -35,6 +34,7 @@ export class AsciiView extends MsgView {
   }
 
   private replaceDelimiter (viewBuffer: Buffer, replaceDelimiter?: number): void {
+    if (this.structure == null) return
     const delimiter = replaceDelimiter ?? this.delimiter
     if (delimiter !== this.writeDelimiter) {
       const structure = this.structure
@@ -55,6 +55,7 @@ export class AsciiView extends MsgView {
   }
 
   public checksum (): number {
+    if (this.structure == null) return -1
     const t = this.getPosition(MsgTag.CheckSum)
     const structure = this.structure
     const prev = structure.tags.tagPos[t - 1]
@@ -127,7 +128,8 @@ export class AsciiView extends MsgView {
       this.writeDelimiter)
   }
 
-  protected stringAtPosition (position: number): string {
+  protected stringAtPosition (position: number): string | null {
+    if (this.structure == null) return null
     const tags = this.structure.tags
     if (position < 0 || position >= tags.nextTagPos) {
       return null
@@ -136,12 +138,14 @@ export class AsciiView extends MsgView {
     return this.buffer.getString(tag.start, tag.start + tag.len)
   }
 
-  private getBuffer (position: number): Buffer {
+  private getBuffer (position: number): Buffer | null {
+    if (this.structure == null) return null
     const tag: TagPos = this.structure.tags.tagPos[position]
     return this.buffer.getBuffer(tag.start, tag.start + tag.len)
   }
 
-  private getNumber (position: number, isFloat: boolean = false): number {
+  private getNumber (position: number, isFloat: boolean = false): number | null {
+    if (this.structure == null) return null
     const buffer = this.buffer
     const tag: TagPos = this.structure.tags.tagPos[position]
     if (isFloat) {
@@ -151,7 +155,8 @@ export class AsciiView extends MsgView {
     }
   }
 
-  private getTime (tag: number, useUtc: boolean): Date {
+  private getTime (tag: number, useUtc: boolean): Date | null {
+    if (this.structure == null) return null
     const formatter = this.timeFormatter
     const position: number = this.getPosition(tag)
     const tagPos: TagPos = this.structure.tags.tagPos[position]
@@ -165,7 +170,8 @@ export class AsciiView extends MsgView {
     }
   }
 
-  private getDate (tag: number, useUtc: boolean): Date {
+  private getDate (tag: number, useUtc: boolean): Date | null {
+    if (this.structure == null) return null
     const formatter = this.timeFormatter
     const position: number = this.getPosition(tag)
     if (position < 0) {
@@ -183,14 +189,15 @@ export class AsciiView extends MsgView {
     }
   }
 
-  private getDateTime (tag: number, useUtc: boolean): Date {
+  private getDateTime (tag: number, useUtc: boolean): Date | null {
+    if (this.structure == null) return null
     const position: number = this.getPosition(tag)
     const tagPos: TagPos = this.structure.tags.tagPos[position]
     const formatter = this.timeFormatter
     if (position < 0) {
       return null
     }
-        // (SendingTime) = 20150417-01:00:08.201
+    // (SendingTime) = 20150417-01:00:08.201
     if (tagPos.len < 8) {
       return null
     }
@@ -202,7 +209,8 @@ export class AsciiView extends MsgView {
     }
   }
 
-  private getBoolean (tag: number): boolean {
+  private getBoolean (tag: number): boolean | null {
+    if (this.structure == null) return null
     const position: number = this.getPosition(tag)
     if (position < 0) {
       return null

package/src/buffer/ascii/itime-formatter.ts

@@ -1,19 +1,19 @@
 export interface ITimeFormatter {
-  writeLocalDate (v: Date): void
-  writeUtcDate (v: Date): void
+  writeLocalDate: (v: Date) => void
+  writeUtcDate: (v: Date) => void
 
-  writeLocalTimestamp (v: Date): void
-  writeUtcTimestamp (v: Date): void
+  writeLocalTimestamp: (v: Date) => void
+  writeUtcTimestamp: (v: Date) => void
 
-  writeLocalTime (v: Date): void
-  writeUtcTime (v: Date): void
+  writeLocalTime: (v: Date) => void
+  writeUtcTime: (v: Date) => void
 
-  getLocalDate (start: number): Date
-  getUtcDate (start: number): Date
+  getLocalDate: (start: number) => Date | null
+  getUtcDate: (start: number) => Date | null
 
-  getLocalTime (start: number): Date
-  getUtcTime (start: number): Date
+  getLocalTime: (start: number) => Date | null
+  getUtcTime: (start: number) => Date | null
 
-  getLocalTimestamp (start: number, end: number): Date
-  getUtcTimestamp (start: number, end: number): Date
+  getLocalTimestamp: (start: number, end: number) => Date | null
+  getUtcTimestamp: (start: number, end: number) => Date | null
 }