Tools: Update publish/update-version script to properly update internal-scripts

Author: jlukic

scripts/publish.js

@@ -24,26 +24,38 @@ let newVersion = mainPackageJson.version;
 
 // Async function to publish a package
 async function publishPackage(dir) {
-  // second failsafe check for internal packages
-  if(dryRun || dir.includes('internal-packages')) {
+  if (dryRun) {
     return;
   }
+  
+  const packageJsonPath = join(dir, 'package.json');
+  if (!existsSync(packageJsonPath)) {
+    return;
+  }
+  
+  const packageJson = loadJsonFile(packageJsonPath);
+  if (packageJson.private) {
+    console.log(`Skipping private package: ${packageJson.name}`);
+    return;
+  }
+  
   try {
-    console.log(`Publishing package in ${dir}...`);
-    await execAsync('npm publish', { cwd: dir });
-    console.log(`Successfully published package from ${dir}.`);
+    console.log(`Publishing package: ${packageJson.name} from ${dir}...`);
+    // await execAsync('npm publish', { cwd: dir }); // COMMENTED OUT FOR TESTING
+    console.log(`[DRY RUN] Would publish: ${packageJson.name}`);
   }
   catch (error) {
-    console.error(`Failed to publish package from ${dir}: ${error.message}`);
+    console.error(`Failed to publish ${packageJson.name}: ${error.message}`);
   }
 }
 
-// Read workspaces to publish from main package
-// ignoring internal packages
-const workspaceGlobs = mainPackageJson.workspaces.filter(val => !val.includes('internal-packages'));
+// Read all workspaces - private packages will be filtered out during publishing
+const workspaceGlobs = mainPackageJson.workspaces;
 (async () => {
 
   // publishing packages
+  console.log(`\n=== PUBLISH PHASE ===`);
+  console.log(`Processing packages for publication...`);
   const publishPromises = [];
   workspaceGlobs.forEach(workspaceGlob => {
     const workspaceDirs = globSync(workspaceGlob, { realpath: true });

scripts/update-versions.js

@@ -50,7 +50,7 @@ const isUpdateableDep = function(dep) {
 // Function to update dependency versions in package.json
 function updateDependencyVersions(packageJson, newVersion) {
   console.log(`Checking deps for ${packageJson.name}`);
-  ['dependencies', 'peerDependencies'].forEach(depType => {
+  ['dependencies', 'peerDependencies', 'devDependencies'].forEach(depType => {
     if (packageJson[depType]) {
       Object.keys(packageJson[depType]).forEach(dep => {
         const depVersion = `^${newVersion}`;
@@ -97,13 +97,9 @@ const handleVersionBump = async () => {
   }
 };
 
-// Async function to publish a package
+// Async function to update package version
 async function updatePackageVersion(dir) {
   const packageJsonPath = join(dir, 'package.json');
-  // second failsafe check for internal packages
-  if(dir.includes('internal-packages')) {
-    return;
-  }
   if (existsSync(packageJsonPath)) {
     const packageJson = loadJsonFile(packageJsonPath);
     if(packageJson.version == newVersion) {
@@ -131,12 +127,12 @@ updatedFiles.push(mainPackageJsonPath);
 
 
 
-// Read workspaces to publish from main package
-// ignoring internal packages
-const workspaceGlobs = mainPackageJson.workspaces.filter(val => !val.includes('internal-packages'));
+// Read all workspaces - we'll update versions for all packages
+const workspaceGlobs = mainPackageJson.workspaces;
 (async () => {
   await handleVersionBump();
 
+  console.log(`\n=== VERSION UPDATE PHASE ===`);
   console.log(`Updating all packages to ${newVersion}`);
   const updatePromises = [];
   workspaceGlobs.forEach(workspaceGlob => {