Reduce pkg/cli package manifest largefunc backlog

[Copilot]

Daily custom lint flagged oversized pkg/cli helpers. This PR takes a focused first slice by reducing the add_package_manifest.go function-length findings without broad rewrites.

• Repository package resolution

  • Extracts repo slug parsing, ref resolution, extension-file resolution, and result construction from resolveRepositoryPackage.

• Manifest parsing

  • Splits YAML/root validation, version checks, and metadata population out of parseRepositoryPackageManifest.

• Skill file resolution

  • Separates skill directory normalization, manifest/auto-scan merging, marker validation, and recursive file listing.

manifest, warnings, err := parseRepositoryPackageManifest(manifestPath, manifestContent)
installationSources, includeSkillDirs, includeAgentFiles, err :=
	resolveRepositoryPackageInstallablePaths(ctx, owner, repo, packagePath, ref, host, manifest, manifestPath)
extensionFiles, err := resolveRepositoryPackageExtensionFiles(ctx, repositoryPackageExtensionFilesRequest{...})

• Fixes [lint-monster] reduce pkg/cli function-length backlog (first slice) #52208

[github-actions]

Triage: pr-type:refactor · pr-risk:low · Score 35/100 (impact 10, urgency 10, quality 15)

Recommended action: batch_review (batch: lintmonster-refactor)

Function-length reduction in add_package_manifest.go (1 file, +217/-108). Single-file, mechanical extraction. Draft, CI not yet run. Grouped with #52220, #52219, #52212.

Generated by 🔧 PR Triage Agent · auto · 44.6 AIC · ⌖ 2.44 AIC · ⊞ 7.8K · ◷

[github-actions]

Triage

• Category: refactor
• Risk: low
• Priority score: 36/100 (impact 15, urgency 9, quality 12)
• Action: batch_review
• Batch: lintmonster-refactor (with Refactor duplicate console format helpers and unify integer tag dispatch; add MCPServerID validity API #52220, Reduce pkg/workflow largefunc backlog via frontmatter parser helper extraction #52219, Refactor nested YAML value extraction #52212)

Largest diff in the batch (217 additions) targeting add_package_manifest.go largefunc backlog. CI unstable — recheck before merge.

Generated by 🔧 PR Triage Agent · auto · 41.1 AIC · ⌖ 2.53 AIC · ⊞ 7.8K · ◷