gtx-cli 2.5.34-alpha.0 → 2.5.35

package/dist/react/jsx/utils/parseStringFunction.js

@@ -1,6 +1,6 @@
 import * as t from '@babel/types';
 import { isStaticExpression, isValidIcu } from '../evaluateJsx.js';
-import { GT_ATTRIBUTES_WITH_SUGAR, MSG_TRANSLATION_FUNCTION, INLINE_TRANSLATION_HOOK, INLINE_TRANSLATION_HOOK_ASYNC, mapAttributeName, INLINE_MESSAGE_HOOK, INLINE_MESSAGE_HOOK_ASYNC, } from './constants.js';
+import { GT_ATTRIBUTES_WITH_SUGAR, MSG_TRANSLATION_HOOK, INLINE_TRANSLATION_HOOK, INLINE_TRANSLATION_HOOK_ASYNC, mapAttributeName, INLINE_MESSAGE_HOOK, INLINE_MESSAGE_HOOK_ASYNC, } from './constants.js';
 import { warnNonStaticExpressionSync, warnNonStringSync, warnTemplateLiteralSync, warnAsyncUseGT, warnSyncGetGT, warnInvalidIcuSync, warnInvalidMaxCharsSync, } from '../../../console/index.js';
 import generateModule from '@babel/generator';
 import traverseModule from '@babel/traverse';
@@ -12,10 +12,7 @@ import pathModule from 'node:path';
 import { parse } from '@babel/parser';
 import { resolveImportPath } from './resolveImportPath.js';
 import { buildImportMap } from './buildImportMap.js';
-import { handleStaticExpression } from './parseDeclareStatic.js';
-import { nodeToStrings } from './parseString.js';
 import { isNumberLiteral } from './isNumberLiteral.js';
-import { indexVars } from 'generaltranslation/internal';
 /**
  * Cache for resolved import paths to avoid redundant I/O operations.
  * Key: `${currentFile}::${importPath}`
@@ -45,62 +42,11 @@ export function clearParsingCaches() {
  * - Metadata extraction from options object
  * - Error reporting for non-static expressions and template literals with expressions
  */
-function processTranslationCall(tPath, updates, errors, warnings, file, ignoreAdditionalData, ignoreDynamicContent, ignoreInvalidIcu, parsingOptions) {
+function processTranslationCall(tPath, updates, errors, warnings, file, ignoreAdditionalData, ignoreDynamicContent, ignoreInvalidIcu) {
     if (tPath.parent.type === 'CallExpression' &&
         tPath.parent.arguments.length > 0) {
         const arg = tPath.parent.arguments[0];
-        // if (t.isExpression(arg)) {
-        if (!ignoreDynamicContent &&
-            t.isExpression(arg) &&
-            !isStaticExpression(arg).isStatic) {
-            const result = handleStaticExpression(arg, tPath, file, parsingOptions, errors);
-            if (result) {
-                const strings = nodeToStrings(result).map(indexVars);
-                if (!ignoreInvalidIcu) {
-                    for (const string of strings) {
-                        const { isValid, error } = isValidIcu(string);
-                        if (!isValid) {
-                            warnings.add(warnInvalidIcuSync(file, string, error ?? 'Unknown error', `${arg.loc?.start?.line}:${arg.loc?.start?.column}`));
-                            return;
-                        }
-                    }
-                }
-                // get metadata and id from options
-                const options = tPath.parent.arguments[1];
-                const metadata = {};
-                if (options && options.type === 'ObjectExpression') {
-                    options.properties.forEach((prop) => {
-                        if (prop.type === 'ObjectProperty' &&
-                            prop.key.type === 'Identifier') {
-                            const attribute = prop.key.name;
-                            if (GT_ATTRIBUTES_WITH_SUGAR.includes(attribute) &&
-                                t.isExpression(prop.value)) {
-                                const result = isStaticExpression(prop.value);
-                                if (!result.isStatic) {
-                                    errors.push(warnNonStaticExpressionSync(file, attribute, generate(prop.value).code, `${prop.loc?.start?.line}:${prop.loc?.start?.column}`));
-                                }
-                                if (result.isStatic && result.value && !ignoreAdditionalData) {
-                                    // Map $id and $context to id and context
-                                    metadata[mapAttributeName(attribute)] = result.value;
-                                }
-                            }
-                        }
-                    });
-                }
-                for (const string of strings) {
-                    updates.push({
-                        dataFormat: 'ICU',
-                        source: string,
-                        metadata: { ...metadata },
-                    });
-                }
-                return;
-            }
-            // Nothing returned, push error
-            errors.push(warnNonStringSync(file, generate(arg).code, `${arg.loc?.start?.line}:${arg.loc?.start?.column}`));
-            // ignore dynamic content flag is triggered, check strings are valid ICU
-        }
-        else if (arg.type === 'StringLiteral' ||
+        if (arg.type === 'StringLiteral' ||
             (t.isTemplateLiteral(arg) && arg.expressions.length === 0)) {
             const source = arg.type === 'StringLiteral' ? arg.value : arg.quasis[0].value.raw;
             // Validate is ICU
@@ -125,7 +71,9 @@ function processTranslationCall(tPath, updates, errors, warnings, file, ignoreAd
                             if (!result.isStatic) {
                                 errors.push(warnNonStaticExpressionSync(file, attribute, generate(prop.value).code, `${prop.loc?.start?.line}:${prop.loc?.start?.column}`));
                             }
-                            if (result.isStatic && result.value && !ignoreAdditionalData) {
+                            if (result.isStatic &&
+                                result.value != null &&
+                                !ignoreAdditionalData) {
                                 const mappedKey = mapAttributeName(attribute);
                                 if (attribute === '$maxChars') {
                                     if ((typeof result.value === 'string' &&
@@ -176,7 +124,6 @@ function processTranslationCall(tPath, updates, errors, warnings, file, ignoreAd
                 errors.push(warnNonStringSync(file, generate(arg).code, `${arg.loc?.start?.line}:${arg.loc?.start?.column}`));
             }
         }
-        // }
     }
 }
 /**
@@ -201,7 +148,7 @@ function extractParameterName(param) {
  * @param visited Set to track already visited variables to prevent infinite loops
  * @returns Array of all variable names that reference the original translation callback
  */
-export function resolveVariableAliases(scope, variableName, visited = new Set()) {
+function resolveVariableAliases(scope, variableName, visited = new Set()) {
     if (visited.has(variableName)) {
         return []; // Prevent infinite loops
     }
@@ -237,7 +184,7 @@ function handleFunctionCall(tPath, updates, errors, warnings, file, importMap, i
     if (tPath.parent.type === 'CallExpression' &&
         tPath.parent.callee === tPath.node) {
         // Direct translation call: t('hello')
-        processTranslationCall(tPath, updates, errors, warnings, file, ignoreAdditionalData, ignoreDynamicContent, ignoreInvalidIcu, parsingOptions);
+        processTranslationCall(tPath, updates, errors, warnings, file, ignoreAdditionalData, ignoreDynamicContent, ignoreInvalidIcu);
     }
     else if (tPath.parent.type === 'CallExpression' &&
         t.isExpression(tPath.node) &&
@@ -426,14 +373,14 @@ export function parseStrings(importName, originalName, path, updates, errors, wa
     const referencePaths = path.scope.bindings[importName]?.referencePaths || [];
     for (const refPath of referencePaths) {
         // Handle msg() calls directly without variable assignment
-        if (originalName === MSG_TRANSLATION_FUNCTION) {
+        if (originalName === MSG_TRANSLATION_HOOK) {
             const ignoreAdditionalData = false;
             const ignoreDynamicContent = false;
             const ignoreInvalidIcu = false;
             // Check if this is a direct call to msg('string')
             if (refPath.parent.type === 'CallExpression' &&
                 refPath.parent.callee === refPath.node) {
-                processTranslationCall(refPath, updates, errors, warnings, file, ignoreAdditionalData, ignoreDynamicContent, ignoreInvalidIcu, parsingOptions);
+                processTranslationCall(refPath, updates, errors, warnings, file, ignoreAdditionalData, ignoreDynamicContent, ignoreInvalidIcu);
             }
             continue;
         }

package/dist/workflow/download.js

@@ -70,7 +70,6 @@ export async function downloadTranslations(fileVersionData, jobData, branchData,
             logger.error(`${chalk.red(`${pollResult.fileTracker.failed.size} file(s) failed to translate:`)}\n${Array.from(pollResult.fileTracker.failed.entries())
                 .map(([key, value]) => `- ${value.fileName}`)
                 .join('\n')}`);
-            return false;
         }
         if (!pollResult.success) {
             return false;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gtx-cli",
-  "version": "2.5.34-alpha.0",
+  "version": "2.5.35",
   "main": "dist/index.js",
   "bin": "dist/main.js",
   "files": [
@@ -106,7 +106,7 @@
     "unified": "^11.0.5",
     "unist-util-visit": "^5.0.0",
     "yaml": "^2.8.0",
-    "generaltranslation": "8.1.4-alpha.0"
+    "generaltranslation": "8.1.3"
   },
   "devDependencies": {
     "@babel/types": "^7.28.4",

package/dist/console/formatting.d.ts

@@ -1 +0,0 @@
-export declare function formatCodeClamp(code: string): string;

package/dist/console/formatting.js

@@ -1,7 +0,0 @@
-const MAX_CODE_GEN_LENGTH = 100;
-export function formatCodeClamp(code) {
-    if (code.length <= MAX_CODE_GEN_LENGTH) {
-        return code;
-    }
-    return `${code.slice(0, MAX_CODE_GEN_LENGTH)}...`;
-}

package/dist/react/jsx/utils/getCalleeNameFromExpression.d.ts

@@ -1,9 +0,0 @@
-import * as t from '@babel/types';
-/**
- * Get the callee name from an expression: ... = useGT();
- * Rule of thumb, only call on expressions with parentheses
- */
-export declare function getCalleeNameFromExpression(expr: t.Expression): {
-    namespaceName: string | null;
-    functionName: string | null;
-};

package/dist/react/jsx/utils/getCalleeNameFromExpression.js

@@ -1,32 +0,0 @@
-import * as t from '@babel/types';
-/**
- * Get the callee name from an expression: ... = useGT();
- * Rule of thumb, only call on expressions with parentheses
- */
-export function getCalleeNameFromExpression(expr) {
-    // If its an await expression, unwrap it
-    if (t.isAwaitExpression(expr)) {
-        return getCalleeNameFromExpression(expr.argument);
-    }
-    // Check that this is a call expression eg: func()
-    if (!t.isCallExpression(expr)) {
-        return { namespaceName: null, functionName: null };
-    }
-    // Get the callee name
-    const calleeName = expr.callee;
-    // Simple case: ... = useGT();
-    if (t.isIdentifier(calleeName)) {
-        return { namespaceName: null, functionName: calleeName.name };
-    }
-    // Member expression: ... = GT.useGT();
-    if (t.isMemberExpression(calleeName)) {
-        if (t.isIdentifier(calleeName.object) &&
-            t.isIdentifier(calleeName.property)) {
-            return {
-                namespaceName: calleeName.object.name,
-                functionName: calleeName.property.name,
-            };
-        }
-    }
-    return { namespaceName: null, functionName: null };
-}

package/dist/react/jsx/utils/parseDeclareStatic.d.ts

@@ -1,15 +0,0 @@
-import * as t from '@babel/types';
-import { NodePath } from '@babel/traverse';
-import { ParsingConfigOptions } from '../../../types/parsing.js';
-import { StringNode } from './types.js';
-export type StringTree = (string | StringTree)[];
-/**
- * Checks if an expression is static or uses declareStatic
- * Returns a Node representing the parsed expression
- * @param expr - The expression to check
- * @param tPath - NodePath for scope resolution
- * @param file - Current file path
- * @param parsingOptions - Parsing configuration
- * @returns Node | null - The parsed node, or null if invalid
- */
-export declare function handleStaticExpression(expr: t.Expression, tPath: NodePath, file: string, parsingOptions: ParsingConfigOptions, errors: string[]): StringNode | null;