Add Parsing Functionality

[jorinvo]

So far ratlog.js only provides logging. The library is so simple that I think it is fine to add parsing to the same package.
This could simply be a function like:

import { parseRatlog } from 'ratlog'

const logs = parseRatlog(stringOfOneOrMoreLogLines)
assert(logs[0] === { message: 'hi', tags: ['tag'], fields: { field: 'value' } })

[legowerewolf]

Current usage of standard formatter blocks implementation using regex lookbehind, even with ignore comments. @jorinvo, will you fix?

[jorinvo]

hi @legowerewolf, sure, feel free to change that. The only thing I want from formatting is to not have to care about it 😉
Also, great to see you picked up the issue, thanks :)

[legowerewolf]

I've finished, I think, except there's some test cases that're still causing problems. Can you verify that this is valid?

{
  "log": "count | count 1\n",
  "data": {
      "message": "count| count 1"
  }
}

Writing the data as provided results in count\| count 1

How I read the spec, the data for that log should be:

{
  "log": "count | count 1\n",
  "data": {
    "message": "count",
    "fields": {
      "count 1": null
    }
  }
}

EDIT:
And this one, too? It's my understanding that the pipe shouldn't appear in the parsed message if the field is valid.

{
  "log": "hi |  \n",
  "data": {
    "message": "hi |",
    "fields": {
      " ": null
    }
  }
}

[jorinvo]

Looks right! Should fix the specs for this :) Thanks for spotting!

[legowerewolf]

Do you want to fix the spec for it, or shall I?

[jorinvo]

Happy if you like to open a pull request :)
As far as I know no one has used the parsing part of the spec data yet.
In case you see any other errors or have another example that would be good to add, happy to extend it.