Watch out if composer update keeps replacing a dependency

Recently, while working on the codebase for this very site, I tried running composer update to upgrade from Drupal 8.8.4 to 8.8.5. Apparently I did this at just the wrong time, as there was an issue with Drupal's dependencies in 8.9.x-dev which caused it to be selected as the upgrade candidate, and the default drupal/core-recommended Composer setting was to allow dev stability, so my site got updated to 8.9.x-dev, which was a bit of a surprise.

"No worries," I thought, "I use git, so I'm protected!" A git reset later, then change my composer.json to use "minimum-stability": "stable", and all is well with the world, right?

Well, no. You see, the problem is Drupal 8.9.x changed from an abandoned package, zendframework/zend-diactoros, to a new package, laminas/laminas-diactoros, that replaces the abandoned package.

When Composer ran the 8.9.x upgrade, it deleted the zendframework/zend-diactoros library from my local vendor folder, and replaced it with laminas/laminas-diactoros. And thus, a frustrating cycle was initiated.

The next time I tried doing a composer update, Drupal core was upgraded to 8.8.5... but I noticed my composer.lock file switched, again, to laminas/laminas-diactoros. And this is bad, because when I deployed this update to my test environment, the environment exploded, with the message:

In DiactorosFactory.php line 37:
                                                                
  Zend Diactoros must be installed to use the DiactorosFactory.

Drush wouldn't work. Drupal wouldn't load pages. I couldn't clear caches (drush, Drupal, or anything).

So then I reverted the composer.lock file changes to the previous commit (with Drupal 8.8.4), and pushed the update to my test server. After running composer install --no-dev, I got the exact same error! How is this possible? The composer.lock file doesn't even list the laminas/laminas-diactoros dependency, and yet, if I check the vendor folder, it's in there—and zendframework/zend-diactoros is not!

Well, I asked about this in the Drupal Slack #composer channel, and a few kind folks like alexpott, greg.1.anderson, and longwave mentioned that Composer doesn't actually use the composer.lock file as the source of truth if you already have dependencies present in the vendor directory.

This revelation blew my mind! I know in the past there has been a time or two when I've blown away the vendor directory because I accidentally messed things up badly. But those were my fault. In this case, I thought composer would use what's in the lock file as the source of truth when installing dependencies, but that is not the case. If there's anything in the vendor directory that says it replaces a package that's in composer.lock, then the package in composer.lock will not be installed.

So the solution? Delete the vendor directory entirely. Then run composer update. To help prevent these kinds of issues in the future, I think my future local environment workflow will be to do an entire git clean of my local repo from time to time (certainly before running any composer operations) to make sure nothing's in the vendor directory that can influence what composer does.

Apparently this behavior will be corrected in Composer 2.0 (though I couldn't find the issue/PR that fixes the issue explicitly to verify). Hopefully it will bring a little more sanity to my life!

Comments

I also came across this problem. I had to manually add the package. This is wrong, but it happened.

This story should be entitled "Never commit the vendor directory" ;)

I haven't, and don't. But yes, still not a good idea to commit it except in rare and unusual circumstances (or maybe in a 'build' branch that's built off a clean environment).

Thank you for so kindly sharing these Composer oddities, both in earlier blog posts, as well as in your migration videos. Since it can stump even a seasoned developer and Drupal expert, imagine a Drupal newbie encountering something like this. Pretty soon they will probably look for another CMS.

The Drupal community should focus on getting "Proposal: Composer Support in Core initiative" completed. Last update was November 2019, more than 7 months ago: https://www.drupal.org/project/ideas/issues/2958021

This happened to me before, I didn't know why but I always delete the vendor after messing with the composer.json or composer.lock to avoid this kind of issues. Thanks for sharing!

We also ran into this today ... it turned out that although there is a new laminas/laminas-zendframework-bridge package, which should solve the missing library issue by adding its own autoloader, if you're using composer < 1.10 and using `composer install --no-dev -o` to make your build (like we were), there was a bug stopping that autoloader from working! So if you upgrade to composer >= 1.10, the laminas packages are fine to use after all!

Ran into the same problem. I was running Composer 1.9.0 and building an artifact via `composer install --no-dev`, and ACSF blew up in fantastic fashion. Tracked down this article, updated to 1.10, and repeated the same build steps. The diff of the final artifacts from the branch build on 1.9 vs 1.10 did indeed show that `Laminas\\ZendFrameworkBridge\\` was only added after bumping Composer to 1.10.7.

+1 thanks for the tip.

> So the solution? Delete the vendor directory entirely. Then run "composer update"

Many thanks, this fixed the issue for us.

Thanks, this saved me a lot of time after running composerize and installing drush 10.