Include package.json when user specifies inputs

We want to include each workspace's package.json as an input when
the user has configured a smaller set of inputs, because the package.json
defines the actual task turbo will execute for that workspace (via scripts).
If that task definition changes, we want to consider the cache invalid.

We only apply this inclusion when inputs are defined, because if inputs
are _not_ defined, _all_ git-tracked files are included in the hash,
'which will already include the package.json file.

Author: mehulkar

cli/internal/hashing/package_deps_hash.go

@@ -38,18 +38,29 @@ func GetPackageDeps(rootPath turbopath.AbsolutePath, p *PackageDepsOptions) (map
 		}
 		result = gitLsTreeOutput
 	} else {
+
+		// Add in package.json to input patterns because if the `scripts` in
+		// the package.json change (i.e. the tasks that turbo executes), we want
+		// a cache miss, since any existing cache could be invalid.
+		// Note this package.json will be resolved relative to the pkgPath.
+		p.InputPatterns = append(p.InputPatterns, "package.json")
+
 		absoluteFilesToHash, err := globby.GlobFiles(pkgPath.ToStringDuringMigration(), p.InputPatterns, nil)
 		if err != nil {
 			return nil, errors.Wrapf(err, "failed to resolve input globs %v", p.InputPatterns)
 		}
+
 		filesToHash := make([]turbopath.AnchoredSystemPath, len(absoluteFilesToHash))
 		for i, rawPath := range absoluteFilesToHash {
 			relativePathString, err := pkgPath.RelativePathString(rawPath)
+
 			if err != nil {
 				return nil, errors.Wrapf(err, "not relative to package: %v", rawPath)
 			}
+
 			filesToHash[i] = turbopath.AnchoredSystemPathFromUpstream(relativePathString)
 		}
+
 		hashes, err := gitHashObject(turbopath.AbsoluteSystemPathFromUpstream(pkgPath.ToStringDuringMigration()), filesToHash)
 		if err != nil {
 			return nil, errors.Wrap(err, "failed hashing resolved inputs globs")

cli/internal/hashing/package_deps_hash_test.go

@@ -239,24 +239,43 @@ func TestGetPackageDeps(t *testing.T) {
 
 	repoRoot := fs.AbsolutePathFromUpstream(t.TempDir())
 	myPkgDir := repoRoot.Join("my-pkg")
+
+	// create the dir first
+	err := myPkgDir.MkdirAll()
+	assert.NilError(t, err, "CreateDir")
+
+	// create file 1
 	committedFilePath := myPkgDir.Join("committed-file")
-	err := committedFilePath.EnsureDir()
-	assert.NilError(t, err, "EnsureDir")
 	err = committedFilePath.WriteFile([]byte("committed bytes"), 0644)
 	assert.NilError(t, err, "WriteFile")
+
+	// create file 2
 	deletedFilePath := myPkgDir.Join("deleted-file")
 	err = deletedFilePath.WriteFile([]byte("delete-me"), 0644)
 	assert.NilError(t, err, "WriteFile")
+
+	// create file 3
 	nestedPath := myPkgDir.Join("dir", "nested-file")
 	assert.NilError(t, nestedPath.EnsureDir(), "EnsureDir")
 	assert.NilError(t, nestedPath.WriteFile([]byte("nested"), 0644), "WriteFile")
+
+	// create a package.json
+	packageJSONPath := myPkgDir.Join("package.json")
+	err = packageJSONPath.WriteFile([]byte("{}"), 0644)
+	assert.NilError(t, err, "WriteFile")
+
+	// set up git repo and commit all
 	requireGitCmd(t, repoRoot, "init", ".")
 	requireGitCmd(t, repoRoot, "config", "--local", "user.name", "test")
 	requireGitCmd(t, repoRoot, "config", "--local", "user.email", "[email protected]")
 	requireGitCmd(t, repoRoot, "add", ".")
 	requireGitCmd(t, repoRoot, "commit", "-m", "foo")
+
+	// remove a file
 	err = deletedFilePath.Remove()
 	assert.NilError(t, err, "Remove")
+
+	// create another untracked file in git
 	uncommittedFilePath := myPkgDir.Join("uncommitted-file")
 	err = uncommittedFilePath.WriteFile([]byte("uncommitted bytes"), 0644)
 	assert.NilError(t, err, "WriteFile")
@@ -265,25 +284,30 @@ func TestGetPackageDeps(t *testing.T) {
 		opts     *PackageDepsOptions
 		expected map[turbopath.AnchoredUnixPath]string
 	}{
+		// base case. when inputs aren't specified, all files hashes are computed
 		{
 			opts: &PackageDepsOptions{
 				PackagePath: "my-pkg",
 			},
 			expected: map[turbopath.AnchoredUnixPath]string{
 				"committed-file":   "3a29e62ea9ba15c4a4009d1f605d391cdd262033",
 				"uncommitted-file": "4e56ad89387e6379e4e91ddfe9872cf6a72c9976",
+				"package.json":     "9e26dfeeb6e641a33dae4961196235bdb965b21b",
 				"dir/nested-file":  "bfe53d766e64d78f80050b73cd1c88095bc70abb",
 			},
 		},
+		// with inputs, only the specified inputs are hashed
 		{
 			opts: &PackageDepsOptions{
 				PackagePath:   "my-pkg",
 				InputPatterns: []string{"uncommitted-file"},
 			},
 			expected: map[turbopath.AnchoredUnixPath]string{
+				"package.json":     "9e26dfeeb6e641a33dae4961196235bdb965b21b",
 				"uncommitted-file": "4e56ad89387e6379e4e91ddfe9872cf6a72c9976",
 			},
 		},
+		// inputs with glob pattern also works
 		{
 			opts: &PackageDepsOptions{
 				PackagePath:   "my-pkg",
@@ -292,16 +316,19 @@ func TestGetPackageDeps(t *testing.T) {
 			expected: map[turbopath.AnchoredUnixPath]string{
 				"committed-file":   "3a29e62ea9ba15c4a4009d1f605d391cdd262033",
 				"uncommitted-file": "4e56ad89387e6379e4e91ddfe9872cf6a72c9976",
+				"package.json":     "9e26dfeeb6e641a33dae4961196235bdb965b21b",
 				"dir/nested-file":  "bfe53d766e64d78f80050b73cd1c88095bc70abb",
 			},
 		},
+		// inputs with another glob pattern works
 		{
 			opts: &PackageDepsOptions{
 				PackagePath:   "my-pkg",
 				InputPatterns: []string{"**/{uncommitted,committed}-file"},
 			},
 			expected: map[turbopath.AnchoredUnixPath]string{
 				"committed-file":   "3a29e62ea9ba15c4a4009d1f605d391cdd262033",
+				"package.json":     "9e26dfeeb6e641a33dae4961196235bdb965b21b",
 				"uncommitted-file": "4e56ad89387e6379e4e91ddfe9872cf6a72c9976",
 			},
 		},