Skip to content

Support additional package repositories - #142

Closed
t-kalinowski wants to merge 12 commits into
mainfrom
fix-ir-issue-128
Closed

t-kalinowski wants to merge 12 commits into
mainfrom
fix-ir-issue-128

Conversation

@t-kalinowski

@t-kalinowski t-kalinowski commented Jul 28, 2026 •

Copy link
Copy Markdown
Member

Summary

Fixes #128 by letting users add CRAN-like package repositories, such as R-universe, before IR's defaults. This lets pak resolve package refs whose dependencies are published outside CRAN and Bioconductor.

User-facing changes

  • Add repeatable --repo options to run, render, preview, rx, and tool commands, plus repos / ir.repos frontmatter.
  • Keep command-line repositories before frontmatter and default repositories. exclude-newer still dates the defaults; additional repositories remain live.
  • Keep Git repositories as pak package refs, not repository options.

Internal changes

  • Pass pak's ordered effective repository set to renv and preserve multi-URL repository specs.
  • Include repositories in resolution keys, refresh live repositories after 24 hours, and use pak artifact identities to avoid stale same-version builds.
  • Preserve credential-free repository options in installed-tool recovery commands.

@t-kalinowski t-kalinowski changed the title Support-additional-package-repositories Support additional package repositories Jul 28, 2026
@t-kalinowski
t-kalinowski marked this pull request as ready for review July 29, 2026 00:16

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 14ef3ed24b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread driver/resolve.R Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4ee3b8076a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/tool.rs Outdated
Comment thread src/runtime.rs Outdated
# Conflicts:
#	README.md
#	docs/cache.qmd
#	docs/config.qmd
#	docs/index.qmd
#	docs/quarto.qmd
#	docs/run.qmd
#	driver/resolve.R
#	src/quickstart.txt
#	tests/snapshots/quickstart.stdout

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b4985665fc

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread driver/resolve.R Outdated
Comment thread src/tool.rs

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1719ce634a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread driver/resolve.R

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2715a10f04

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread driver/resolve.R
@t-kalinowski

Copy link
Copy Markdown
Member Author

My primary motivation for this feature has been addressed upstream with r-xla/anvl#418. I'll hold off on adding support for additional repos in ir until there is a compelling use-case.

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.

pak fails to resolve github dependencies

1 participant