Display non-KRM files in kpt pkg tree output

[aravindtga]

Summary

Enhances kpt pkg tree to show all package files, not just KRM resources.

Changes

• Walk the package tree to discover non-KRM files and render them in the tree output
• Deduplicate files already rendered as KRM resources (by basename)
• Skip dotfiles and dot-directories (.git/, .hidden)

Example

Before:

  Package "my-app"
  ├── [Kptfile]  Kptfile my-app
  ├── [deployment.yaml]  Deployment nginx
  └── Package "subpkg"
      ├── [Kptfile]  Kptfile subpkg
      └── [service.yaml]  Service my-svc

After:

  Package "my-app"
  ├── [Kptfile]  Kptfile my-app
  ├── [deployment.yaml]  Deployment nginx
  ├── README.md
  └── Package "subpkg"
      ├── [Kptfile]  Kptfile subpkg
      ├── [service.yaml]  Service my-svc
      └── NOTES.txt

Testing

• Manually tested with multiple package structures
• Added unit tests

AI usage disclosure

• AI assistance (Kiro CLI) was used to draft portions of this change, and this PR description.

[netlify]

✅ Deploy Preview for kptdocs ready!

Name	Link
🔨 Latest commit	721cf52
🔍 Latest deploy log	https://app.netlify.com/projects/kptdocs/deploys/6a047bf9dfb0d80009190fc9
😎 Deploy Preview	https://deploy-preview-4518--kptdocs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
🤖 Make changes	Run an agent on this branch

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[Copilot]

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Enhances kpt pkg tree to include non-KRM package files in the rendered tree output (while skipping dotfiles/dot-dirs and avoiding duplicates with KRM-rendered files).

Changes:

• Add filesystem discovery of non-KRM files and pass them into the tree renderer.
• Merge non-KRM “directories” into the tree traversal and render those filenames under each package.
• Add/adjust unit tests to validate listing, sorting, dotfile exclusion, symlink skipping, and deduplication.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 8 comments.

File	Description
thirdparty/cmdconfig/commands/cmdtree/tree.go	Extends tree rendering to merge and print non-KRM filenames under package branches and dedup against rendered KRM resources.
thirdparty/cmdconfig/commands/cmdtree/cmdtree.go	Adds discoverNonKRMFiles that walks the directory tree and supplies non-KRM files to TreeWriter.
thirdparty/cmdconfig/commands/cmdtree/cmdtree_test.go	Updates existing expectation and adds new unit tests for non-KRM listing behavior.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated 3 comments.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (1)

thirdparty/cmdconfig/commands/cmdtree/tree.go:112

• Parent-branch selection uses strings.HasPrefix(pkg, parent) which can incorrectly treat sibling directories as ancestors (e.g., parent="a" matches pkg="ab"), causing the tree to nest packages/dirs under the wrong branch. Consider requiring a path-segment boundary (pkg==parent or strings.HasPrefix(pkg, parent+string(filepath.Separator))) or computing the parent via filepath.Dir(pkg) and walking upward.

	for _, pkg := range allKeys {
		// create a branch for this package -- search for the parent package and create
		// the branch under it -- requires that the keys are sorted
		branch := tree
		for parent, subTree := range treeIndex {
			if strings.HasPrefix(pkg, parent) {
				// found a package whose path is a prefix to our own, use this
				// package if a closer one isn't found

[mozesl-nokia]

If we are changing the copyright, we should move it out of the thirdparty folder.

Does it make sense to do:

// Copyright 2019 The Kubernetes Authors, 2026 The kpt Authors.

? Does it work like that?

[mozesl-nokia]

Same here with the copyright

[mozesl-nokia]

Should probably add a skip reason

[mozesl-nokia]

Same thing with the copyright