Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

expose parsePattern and serializePattern #212

Open
samuelstroschein opened this issue Sep 23, 2024 — with Linear · 2 comments
Open

expose parsePattern and serializePattern #212

samuelstroschein opened this issue Sep 23, 2024 — with Linear · 2 comments

Comments

Copy link
Member

samuelstroschein commented Sep 23, 2024

Context

Apps and plugins write the same parse and serialize pattern logic which can be abstracted with a function.

function parsePattern(value: string): Variant["pattern"] {
	const pattern: Variant["pattern"] = []

	// splits a pattern like "Hello {{name}}!" into an array of parts
	// "hello {{name}}, how are you?" -> ["hello ", "{{name}}", ", how are you?"]
	const parts = value.split(/(\{{.*?\}})/).filter((part) => part !== "")

	for (const part of parts) {
		// it's text
		if ((part.startsWith("{{") && part.endsWith("}}")) === false) {
			pattern.push({ type: "text", value: part })
		}
		// it's an expression (only supporting variables for now)
		else {
			const variableName = part.slice(1, -1)
			pattern.push({ type: "expression", arg: { type: "variable-reference", name: variableName } })
		}
	}

	return pattern
}

Proposal

Add a parsePattern and serializePattern function to the SDK to ease writing apps and plugins.

const pattern = parsePattern(
  "Hello {{username}}", { 
    variableReferencePattern: ["{{", "}}"]
  }
)

console.log(pattern)

[{ type: "text", value: "Hello "}, { type: "variable-reference", name: "username"}]
const serialized = serializePattern(pattern, {
  variableReferencePattern: ["{{", "}}"]
})

console.log(serialized)

"Hello {{username}}"

Additional information

  • note that we previously thought that we need a serialized format to enable an abstracted parse/serialize pattern function. that's not true if we choose a function that take a config.
Copy link

felixhaeberle commented Sep 23, 2024

Cool!

  1. Please account for spaces/empty chars in the variable definition scope with the regex. Additionally, it would be even safer/more robust if people can put in there whatever they want, including full sentences with special characters etc.
  2. I hope to switch out those utility functions of Sherlock for this functionality, please have a look – I guess they are doing the same thing. https://github.com/opral/monorepo/blob/882b3ff9c9c23c865c49b1df0597958ec60e6038/inlang/source-code/ide-extension/src/utilities/messages/query.ts#L4
  3. How does error detection, handling & propagation in case of not successfully parsed patterns look like?

Copy link
Member Author

  1. Noted.
  2. Yes, they are doing the same thing.

3. How does error detection, handling & propagation in case of not successfully parsed patterns look like?

Fallback to string. Users can fix the pattern themselves and apps have reduced complexity

i18next is able to reference other messages. Nice feature but no first class support from inlang. Instead of throwing, we can import the string as is. On export, i18next will reference the other message again.

const pattern = parsePattern("reuse $t(keyDeep.inner)")

console.log(pattern)

[{ type: "text", value: "reuse $t(keyDeep.inner)"} ]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants