Enforce Leading and Trailing double quote in ByteMachine

[jonessha]

What is your idea?

In actual usage of Ruler, ByteMachine stores string values always with a beginning and ending double quote. However, ByteMachine is capable of storing string values that begin and end with any character. This unnecessary flexibility leads to more complexity and special cases, for example, to support a leading or trailing wildcard character for wildcard patterns. We can simplify the code and remove some special cases by updating ByteMachine to either enforce the leading/trailing double quotes itself, or to be the component that adds these double quotes.

Would you be willing to make the change?

Maybe.

Additional context

Discussion arose during: #18

[timbray]

Um, for {"x": [true]} does Ruler actually add quotes? At one point it didn't.

[jonessha]

Oops, right, I'm just talking about string values. Will update description.

[timbray]

Also numbers, right?

[jonessha]

Err, numbers don't get quotes added. For example, just verified there were no quotes added for:

machine.addRule("rule1", "{"x": [123]}");

Is that unexpected?

[timbray]

Oops, I said that backward, I meant that numbers are not "-wrapped. Anyhow, I think we have converged here.