feat(sync): add flag '--uploadTranslations'

Author: Anton Savoskin

src/base.ts

@@ -48,6 +48,9 @@ export abstract class Base extends Command {
       description: 'Provider`s namespace. Required for Locize',
       env: 'NAMESPACE',
     }),
+    uploadTranslations: flags.boolean({
+      description: 'Upload existing translations to provider. Useful for provider migration.',
+    }),
   };
 
   async init() {

src/commands/sync.ts

@@ -1,6 +1,6 @@
 import * as path from 'path';
 
-import { asyncForEach, readFile, showError, showInfo, toHash, writeFile } from '../utils';
+import { asyncForEach, readFile, showError, showInfo, toHash, writeFile, head, tail } from '../utils';
 import { Base } from '../base';
 import { Lokalise } from '../providers/lokalise';
 import { Provider } from '../providers/provider';
@@ -60,10 +60,10 @@ export default class Extract extends Base {
 
   async run() {
     const {
-      flags: { langs, provider, projectId, token, version, namespace },
+      flags: { langs, provider, projectId, token, version, namespace, uploadTranslations },
     } = this.parse(Extract);
     const locales = langs.split(',');
-    const defaultLocale = locales[0];
+    const defaultLocale = head(locales);
 
     const getProvider = () => {
       const providers: { [key: string]: any } = {
@@ -88,12 +88,11 @@ export default class Extract extends Base {
     const newMessages = this.provider.getNewMessages();
     if (newMessages.length > 0) {
       showInfo(`New translation keys: ${newMessages.length}`);
-      if (provider === 'locize') {
-        await asyncForEach(locales, locale =>
+      await this.provider.uploadMessages(newMessages.map(id => this.messages[locales[0]][id]));
+      if (uploadTranslations && locales.length > 1) {
+        await asyncForEach(tail(locales), locale =>
           this.provider!.uploadMessages(newMessages.map(id => this.messages[locale][id]).filter(Boolean), locale),
         );
-      } else {
-        await this.provider.uploadMessages(newMessages.map(id => this.messages[locales[0]][id]), defaultLocale);
       }
     }
   }

src/providers/locize.ts

@@ -44,7 +44,7 @@ export class Locize implements Provider {
     return '';
   }
 
-  async uploadMessages(messages: Message[], locale: string) {
+  async uploadMessages(messages: Message[], locale = this.defaultLocale) {
     const headers = { Authorization: `Bearer ${this.apiKey}`, 'content-type': 'application/json' };
     const body = messages.reduce(
       (acc, { id, message, defaultMessage, description }) => {

src/providers/provider.ts

@@ -3,6 +3,6 @@ import { Message } from '../types';
 export interface Provider {
   getKeys(locales: string[]): Promise<void>;
   getMessage(locale: string, id: string): string;
-  uploadMessages(messages: Message[], locale: string): Promise<void>;
+  uploadMessages(messages: Message[], locale?: string): Promise<void>;
   getNewMessages(): string[];
 }

src/utils.ts

@@ -28,6 +28,8 @@ export const toHash = <T>(array: T[], key: string): { [id: string]: T } =>
     (acc: { [id: string]: T }, data: T, index: number) => ((acc[key ? data[key] : index] = data), acc),
     {},
   );
+export const tail = <T>(arr: T[]) => (arr.length > 1 ? arr.slice(1) : arr);
+export const head = <T>(arr: T[]) => arr[0];
 
 type Options = https.RequestOptions & {
   url: string;

test/commands/sync.test.js

@@ -85,7 +85,7 @@ describe('sync locize', () => {
         .reply(200, locizeKeysJson),
     )
     .stderr()
-    .command(['sync', '--messagesDir', messagesDir, '--provider', 'locize'])
+    .command(['sync', '--messagesDir', messagesDir, '--provider', 'locize', '--uploadTranslations'])
     .it('upload new message to locize', ctx => {
       expect(ctx.stderr).toBe('');
     });