Custom framework: Any way to match a namespace alongside the key for the usage match?

[baptisteArno]

Is your feature related to a specific framework or general for this extension
next-international

Is your feature request related to a problem? Please describe.
I'd like to be able to detect a namespace:

 const scopedT = useScopedI18n('folders')
[...]
scopedT('key')

It won't take folders namespace into consideration with this custom settings:

# .vscode/i18n-ally-custom-framework.yml

usageMatchRegex:
  - "[^\\w\\d]t\\(['\"`]({key})['\"`]"
  - "[^\\w\\d]scopedT\\(['\"`]({key})['\"`]"

Describe the solution you'd like
Maybe adding {namespace} for the regex detection, that would add it as a prefix for the key? Or a namespaceMatchRegex config?

[amannn]

I'd be interested in this too! I guess this is required for #820 (if namespaces are used).

[amannn]

It seems like at least programmatically this should be possible:

i18n-ally/src/frameworks/react-i18next.ts

Lines 82 to 108 in 0ea030d

	// useTranslation
	// https://react.i18next.com/latest/usetranslation-hook#loading-namespaces
	getScopeRange(document: TextDocument): ScopeRange[] | undefined {
	if (!this.languageIds.includes(document.languageId as any))
	return
	const ranges: ScopeRange[] = []
	const text = document.getText()
	// Add smaller local scope overrides first
	// Namespaced prefixed keys already handled by rewriteKeys
	// t('foo', { ns: 'ns1' })
	const regT = /\Wt\([^)]*?ns:\s*['"`](\w+)['"`]/g
	for (const match of text.matchAll(regT)) {
	if (typeof match.index !== 'number')
	continue
	if (match[1]) {
	ranges.push({
	start: match.index,
	end: match.index + match[0].length,
	namespace: match[1],
	})
	}
	}

[aranard]

This feature is added in #926, by including a new scopeRangeRegex property in the custom settings file.

[baptisteArno]

This feature is added in #926, by including a new scopeRangeRegex property in the custom settings file.

That's nice! Unfortunately it won't work properly with that setup:

const t = useI18n()
const scopedT = useScopedI18n('scope')

// Use both t and scopedT

[aranard]

@baptisteArno ah I see - the issue is that you can only specify one regex, not multiple? That seems like it'd be straightforward to add. For my use case I only needed to set one, so didn't think to add the option for multiple.

[wesnolte]

I attempted to work around this by using the regex "or" but it doesn't work for some reason. Any idea why? e.g.

scopeRangeRegex: "(use|get)ScopedI18n\\(\\s*\\[?\\s*['\"](.*?)['"]"