From a664d233f0ee4b340d30a266a9794b0ef7ee4190 Mon Sep 17 00:00:00 2001
From: Anatolios Laskaris <github_me@nahsi.dev>
Date: Tue, 28 Mar 2023 21:46:23 +0300
Subject: [PATCH] chore: Set packages version with action [fixes FLU-290]
 (#293)

* Test setting version with action

* Cleanup
---
 .github/workflows/snapshot.yml |   8 +-
 ci.cjs                         | 167 ---------------------------------
 2 files changed, 2 insertions(+), 173 deletions(-)
 delete mode 100644 ci.cjs

diff --git a/.github/workflows/snapshot.yml b/.github/workflows/snapshot.yml
index b806224a..748cd3fc 100644
--- a/.github/workflows/snapshot.yml
+++ b/.github/workflows/snapshot.yml
@@ -165,18 +165,14 @@ jobs:
           secrets: |
             kv/npm-registry/basicauth/ci token | NODE_AUTH_TOKEN
 
+      - run: pnpm -r build
+
       - name: Generate snapshot version
         id: version
         uses: fluencelabs/github-actions/generate-snapshot-id@main
 
-      - name: Set package version
-        run: node ci.cjs bump-version ${{ steps.version.outputs.id }}
-
-      - run: pnpm -r build
-
       - name: Publish snapshots
         id: snapshot
         uses: fluencelabs/github-actions/pnpm-publish-snapshot@main
         with:
           id: ${{ steps.version.outputs.id }}
-          flags: "-r --registry https://npm.fluence.dev --no-git-checks --tag snapshot -filter '@fluencelabs/*'"
diff --git a/ci.cjs b/ci.cjs
deleted file mode 100644
index 6fa69e2c..00000000
--- a/ci.cjs
+++ /dev/null
@@ -1,167 +0,0 @@
-#! /usr/bin/env node
-
-const fs = require("fs").promises;
-const path = require("path");
-
-function printUsage() {
-    console.log(
-        `Usage: "ci check-consistency" or "ci bump-version %postfix%" or "ci get-version"`
-    );
-}
-
-let postfix;
-const mode = process.argv[2];
-
-function validateArgs() {
-    switch (mode) {
-        case "get-version":
-            return true;
-
-        case "bump-version":
-            postfix = process.argv[3];
-            if (!postfix) {
-                printUsage();
-                process.exit();
-            }
-            return true;
-
-        case "":
-        case undefined:
-        case "check-consistency":
-            return true;
-
-        default:
-            return false;
-    }
-}
-
-const PATH_TO_PACKAGES = "./packages/";
-
-async function getPackageJsonsRecursive(currentPath) {
-    return (
-        await Promise.all(
-            (await fs.readdir(currentPath, { withFileTypes: true }))
-                .filter(
-                    (file) =>
-                        file.name !== "node_modules" && file.name !== "@tests" &&
-                        (file.isDirectory() || file.name === "package.json")
-                )
-                .map((file) =>
-                    file.isDirectory()
-                        ? getPackageJsonsRecursive(
-                              path.join(currentPath, file.name)
-                          )
-                        : Promise.resolve([
-                              path.join(process.cwd(), currentPath, file.name),
-                          ])
-                )
-        )
-    ).flat();
-}
-
-async function getVersion(file) {
-    const content = await fs.readFile(file);
-    const json = JSON.parse(content);
-    return [json.name, json.version];
-}
-
-function processDep(obj, name, fn) {
-    if (!obj) {
-        return;
-    }
-
-    if (!obj[name]) {
-        return;
-    }
-
-    fn(obj, obj[name]);
-}
-async function getVersionsMap(allPackageJsons) {
-    return new Map(await Promise.all(allPackageJsons.map(getVersion)));
-}
-
-function getVersionForPackageOrThrow(versionsMap, packageName) {
-    const version = versionsMap.get(packageName);
-    if (!version) {
-        console.log("Failed to get version for package: ", packageName);
-        process.exit(1);
-    }
-    return version;
-}
-
-async function checkConsistency(file, versionsMap) {
-    console.log("Checking: ", file);
-    const content = await fs.readFile(file);
-    const json = JSON.parse(content);
-
-    for (const [name, versionInDep] of versionsMap) {
-        const check = (x, version) => {
-            if (version.includes("*")) {
-                return;
-            }
-
-            if (versionInDep !== version) {
-                console.log(
-                    `Error, versions don't match: ${name}:${version} !== ${versionInDep}`,
-                    file
-                );
-                process.exit(1);
-            }
-        };
-        processDep(json.dependencies, name, check);
-        processDep(json.devDependencies, name, check);
-    }
-}
-
-async function bumpVersions(file, versionsMap) {
-    console.log("Updating: ", file);
-    const content = await fs.readFile(file);
-    const json = JSON.parse(content);
-
-    // bump dependencies
-    for (const [name, version] of versionsMap) {
-        const update = (x) => (x[name] = `${version}-${postfix}`);
-        processDep(json.dependencies, name, update);
-        processDep(json.devDependencies, name, update);
-    }
-
-    // also bump version in package itself
-    const version = getVersionForPackageOrThrow(versionsMap, json.name);
-    json.version = `${version}-${postfix}`;
-
-    const newContent = JSON.stringify(json, undefined, 4) + "\n";
-    await fs.writeFile(file, newContent);
-}
-
-async function processPackageJsons(allPackageJsons, versionsMap, fn) {
-    await Promise.all(allPackageJsons.map((x) => fn(x, versionsMap)));
-}
-
-async function run() {
-    if (!validateArgs()) {
-        printUsage();
-        process.exit(0);
-    }
-
-    const packageJsons = await getPackageJsonsRecursive(PATH_TO_PACKAGES);
-    const versionsMap = await getVersionsMap(packageJsons);
-
-    if (mode === "get-version") {
-        const fjs = versionsMap.get("@fluencelabs/fluence");
-        console.log(fjs);
-        return;
-    }
-
-    // always check consistency
-    console.log("Checking versions consistency...");
-    await processPackageJsons(packageJsons, versionsMap, checkConsistency);
-    console.log("Versions are consistent");
-
-    if (mode === "bump-version") {
-        console.log("Adding postfix: ", postfix);
-        await processPackageJsons(packageJsons, versionsMap, bumpVersions);
-        console.log("Done");
-    }
-}
-
-run();