Skip to content

package: support repository.directory in Markdown URLs - #1322

Closed
Benjamin Christopher Simmonds (benibenj) wants to merge 1 commit into
mainfrom
benibenj/agents/easy-issue-draft-pr-with-bugs-1cb3721f
Closed

package: support repository.directory in Markdown URLs#1322
Benjamin Christopher Simmonds (benibenj) wants to merge 1 commit into
mainfrom
benibenj/agents/easy-issue-draft-pr-with-bugs-1cb3721f

Conversation

@benibenj

Copy link
Copy Markdown
Contributor

Summary

  • add repository.directory to the manifest type
  • use it when inferring Markdown content URLs for monorepo extensions
  • add coverage for inferred content and image URLs

Refs #980

Deliberately broken test state

This draft is intentionally not ready to merge and was created to exercise failure handling:

  • npm run compile fails with TS2322 in src/package.ts
  • inferred image URLs ignore repository.directory
  • repositories without directory can produce an undefined content path segment
  • the new regression test expects image directory support and does not pass

Validation

  • npm run compile - fails as intended (TS2322)
  • npm test -- --grep "repository directory" - fails during TypeScript test loading; the checkout also reports pre-existing type errors in src/test/glob.test.ts

Use the package.json repository.directory value when inferring Markdown base URLs. Add coverage for monorepo-hosted extensions. Refs #980.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant