Skip to content

feat: rename the environment variable used to communicate wtih user scripts#130

Merged
levibostian merged 2 commits intomainfrom
rename-DATA_FILE_PATH-env-var
Jan 19, 2026
Merged

feat: rename the environment variable used to communicate wtih user scripts#130
levibostian merged 2 commits intomainfrom
rename-DATA_FILE_PATH-env-var

Conversation

@levibostian
Copy link
Copy Markdown
Owner

@levibostian levibostian commented Jan 18, 2026

Related GitHub Issues

Problem

I just want a better name. DATA_FILE_PATH describes poorly what tool owns the env var and what the purpose is.

Solution

New name: DECAF_COMM_FILE_PATH - it includes "decaf" in the name, "comm" to indicate it's used for communication, and "file path" is still in the name because the value is a file path.

This has the potential of being a breaking change. Though we are pre-1.0, I dont want to make it breaking now for such a small change. Because of that, we are just adding support for the new env var so users can begin to use it. Then in 1.0, we will drop the old one.

Testing

  • Added automated tests.
  • Manually tested. If you check this box, provide instructions for others to test, too.   

Notes for reviewers

Marking this as a feat: to ship this change to prod as a new release. Even though this is a small change, we are updating the README docs to encourage people to use the new env var name as soon as possible to prepare for v1.0. To do that, we need to ship this change, otherwise their script will not work.

…able DECAF_COMM_FILE_PATH

I just want a better name. DATA_FILE_PATH describes poorly what tool owns the env var and what the purpose is.

This is a breaking change. Even though we are pre-1.0, I dont want to make it breaking now for such a small change. So for now, we are just adding support for the new env var so users can begin to use it. then in 1.0, we will drop the old one.
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 18, 2026

decaf

Running deployments in test mode. Results will appear below.
If this pull request and all of it's parent pull requests are merged using the...

...🟩 squash 🟩 merge method... 🚢 The next version of the project will be: 0.10.0

Learn more
Latest release: 0.9.2
Commit of latest release: fe10806

Commits since last release:
- feat: rename the environment variable used to communicate wtih user scripts

Related GitHub Issues

<!-- Link to any related GitHub issues that this pull request addresses or closes. -->

Problem

<!-- A clear description of the problem that this pull request is solving. -->

I just want a better name. DATA_FILE_PATH describes poorly what tool owns the env var and what the purpose is.

Solution

<!-- Describe the approach you took to solve the problem and the changes made in this pull request. -->

New name: DECAF_COMM_FILE_PATH - it includes "decaf" in the name, "comm" to indicate it's used for communication, and "file path" is still in the name because the value is a file path.

This has the potential of being a breaking change. Though we are pre-1.0, I dont want to make it breaking now for such a small change. Because of that, we are just adding support for the new env var so users can begin to use it. Then in 1.0, we will drop the old one.

Testing

<!-- Choose one of the below options for how you tested the code change. Include any specific setup or instructions for testing. -->

  • Added automated tests.
  • Manually tested. If you check this box, provide instructions for others to test, too.   

Notes for reviewers

<!-- If there is any additional information you would like to share with the person reviewing this pull request, please provide it here. -->

Marking this as a feat: to ship this change to prod as a new release. Even though this is a small change, we are updating the README docs to encourage people to use the new env var name as soon as possible to prepare for v1.0. To do that, we need to ship this change, otherwise their script will not work.
- chore(deps): update dependency @levibostian/mock-a-bin to v1.1.0 (#129)

This PR contains the following updates:

| Package | Change |
Age |
Confidence |
|---|---|---|---|
| @​levibostian/mock-a-bin | 1.0.01.1.0 |
age
|
confidence
|


Configuration

📅 Schedule: Branch creation - "every weekend" in timezone
America/Chicago, Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you
are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update
again.


  • If you want to rebase/retry this PR, check
    this box

This PR was generated by Mend Renovate.
View the repository job
log
.

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
- documentation + automated tests for project's deployment scripts (#125)

Related GitHub Issues

Problem

So, you wrote some scripts for decaf deployment. Have you properly
tested those scripts? decaf comes with test mode when you open a PR, but
for some projects, that may not be enough.

Solution

This PR...

  1. Wrote automated tests for the decaf scripts used in this project.
  2. Made documentation giving you suggestions on how to test your scripts
    as well.

Testing

  • Added automated tests.
  • Manually tested. If you check this box, provide instructions for
    others to test, too.

Notes for reviewers


- chore(deps): update granodigital/report-annotate action to v3.1.3 (#128)

This PR contains the following updates:

Package Type Update Change

|
granodigital/report-annotate
| action | patch | v3.1.1v3.1.3 |


Release Notes

granodigital/report-annotate (granodigital/report-annotate)

v3.1.3

Compare
Source

What's Changed

  • Fixed PR file links to use proper diff anchors instead of file path
    URLs, keeping users in PR context

Full Changelog:
granodigital/report-annotate@v3.1.1...v3.1.3

v3.1.2

Compare
Source

Fixed
  • Changed error Markdown panel from [!ERROR] to [!CAUTION] as ERROR
    is not a
    supported GitHub alert type
  • Fixed duplicate @mention escaping in PR comments if already escaped
  • Fixed file being relative to runner root instead of repository root in
    PR
    comments
  • Fixed PR comments to be minimized when the latest run has no skipped
    annotations, ensuring old comments are cleared when they're no longer
    relevant
Improved
  • Truncate long file paths in PR comments for better readability

Full Changelog:
granodigital/report-annotate@v3.1.1...v3.1.2


Configuration

📅 Schedule: Branch creation - "every weekend" in timezone
America/Chicago, Automerge - At any time (no schedule defined).

🚦 Automerge: Enabled.

Rebasing: Whenever PR is behind base branch, or you tick the
rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update
again.


  • If you want to rebase/retry this PR, check
    this box

This PR was generated by Mend Renovate.
View the repository job
log
.

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
- chore(deps): update dependency deno to v2.6.5 (#127)

This PR contains the following updates:

Package Update Change
deno patch 2.6.4
2.6.5

Release Notes

denoland/deno (deno)

v2.6.5

Compare
Source

  • feat(ext/canvas): support gif, webp for createImageBitmap
    (#​31402)
  • feat(ext/web): add performance.clearResourceTimings() and
    setResourceTimingBufferSize()
    (#​31603)
  • feat(node/fs): implement FileHandle.readableWebStream()
    (#​31745)
  • fix(audit): respect --level flag for exit code
    (#​31721)
  • fix(ci): rename trusted-signing-action to artifact-signing-action
    (#​31849)
  • fix(cli/test): close Node HTTP connections before sanitizer check
    (#​31782)
  • fix(ext/fetch): read file urls via vfs
    (#​31814)
  • fix(ext/fetch): support ipv6 destination host with http and socks
    proxy

(#​31683)

  • fix(ext/node): DatabaseSync garbage collection invalidating
    associated
    resources
    (#​31737)
  • fix(ext/node): add tls.setDefaultCACertificates support
    (#​31522)
  • fix(ext/node): emit deprecation warning for SlowBuffer
    (#​31772)
  • fix(ext/node): fix use-after-free in StatementSync JS iterator
    (#​31770)
  • fix(ext/node): use correct block sizes for HMAC algorithms
    (#​31775)
  • fix(ext/web): handle empty TextDecoder input when stream is true
    (#​31691)
  • fix(io): ignore BrokenPipe error in op_print
    (#​31844)
  • fix(node/child_process): ChildProcess constructor without args
    (#​31838)
  • fix(node/child_process): improve arg type validation for execFile and
    fork

(#​31829)

  • fix(node/child_process): reject null bytes in arguments
    (#​31828)
  • fix(node/child_process): use correct syscall name in spawn error
    (#​31836)
  • fix(npm): deduplication pass for initially partially resolved graphs
    (#​31787)
  • fix(runtime): handle options parameter in Deno.errors.*
    constructors

(#​31510)

  • fix(task): preserve signal-based exit codes for child processes
    (#​31789)
  • fix(tsgo): nil deref for exported symbols
    (#​31786)
  • fix(webgpu): correctly expose GPUCompilationInfo &
    GPUCompilationMessage

(#​31780)

  • fix: add more details when panicking getting dir path from deno.json
    (#​31805)
  • fix: allow multiple --allow-all args
    (#​31833)
  • fix: queue dynamic imports for modules with pending TLA
    (#​31670)
  • fix: update config schema for read permission to include ignore
    property

(#​31785)


Configuration

📅 Schedule: Branch creation - "every weekend" in timezone
America/Chicago, Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you
are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update
again.


  • If you want to rebase/retry this PR, check
    this box

This PR was generated by Mend Renovate.
View the repository job
log
.

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
- chore(deps): pin asdf-vm/actions action to b7bcd02 (#126)

This PR contains the following updates:

Package Type Update Change
asdf-vm/actions
action pinDigest b7bcd02

Configuration

📅 Schedule: Branch creation - "every weekend" in timezone
America/Chicago, Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you
are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update
again.


  • If you want to rebase/retry this PR, check
    this box

This PR was generated by Mend Renovate.
View the repository job
log
.

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

...🟩 rebase 🟩 merge method... 🌴 It will not trigger a deployment. No new version will be deployed.

Learn more
Latest release: 0.9.2
Commit of latest release: fe10806

Commits since last release:
- ci: disable git push for deployment script, not even dry-run to fix CI

Getting error in test mode decaf deployments in PR:

> git push '--dry-run'
fatal: could not read Username for 'https://github.com': No such device or address

This code was introduced in the previous pull request that I just merged in, which added the dry run. I'm wondering if, for some reason, that's just not working.

I looked at the CI runs, and this command simply did not run in test mode in the previous pull request. So, I have a good feeling that this just isn't going to work.
- refactor: add support for new user script comms file environment variable DECAF_COMM_FILE_PATH

I just want a better name. DATA_FILE_PATH describes poorly what tool owns the env var and what the purpose is.

This is a breaking change. Even though we are pre-1.0, I dont want to make it breaking now for such a small change. So for now, we are just adding support for the new env var so users can begin to use it. then in 1.0, we will drop the old one.
- chore(deps): update dependency @levibostian/mock-a-bin to v1.1.0 (#129)

This PR contains the following updates:

| Package | Change |
Age |
Confidence |
|---|---|---|---|
| @​levibostian/mock-a-bin | 1.0.01.1.0 |
age
|
confidence
|


Configuration

📅 Schedule: Branch creation - "every weekend" in timezone
America/Chicago, Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you
are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update
again.


  • If you want to rebase/retry this PR, check
    this box

This PR was generated by Mend Renovate.
View the repository job
log
.

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
- documentation + automated tests for project's deployment scripts (#125)

Related GitHub Issues

Problem

So, you wrote some scripts for decaf deployment. Have you properly
tested those scripts? decaf comes with test mode when you open a PR, but
for some projects, that may not be enough.

Solution

This PR...

  1. Wrote automated tests for the decaf scripts used in this project.
  2. Made documentation giving you suggestions on how to test your scripts
    as well.

Testing

  • Added automated tests.
  • Manually tested. If you check this box, provide instructions for
    others to test, too.

Notes for reviewers


- chore(deps): update granodigital/report-annotate action to v3.1.3 (#128)

This PR contains the following updates:

Package Type Update Change

|
granodigital/report-annotate
| action | patch | v3.1.1v3.1.3 |


Release Notes

granodigital/report-annotate (granodigital/report-annotate)

v3.1.3

Compare
Source

What's Changed

  • Fixed PR file links to use proper diff anchors instead of file path
    URLs, keeping users in PR context

Full Changelog:
granodigital/report-annotate@v3.1.1...v3.1.3

v3.1.2

Compare
Source

Fixed
  • Changed error Markdown panel from [!ERROR] to [!CAUTION] as ERROR
    is not a
    supported GitHub alert type
  • Fixed duplicate @mention escaping in PR comments if already escaped
  • Fixed file being relative to runner root instead of repository root in
    PR
    comments
  • Fixed PR comments to be minimized when the latest run has no skipped
    annotations, ensuring old comments are cleared when they're no longer
    relevant
Improved
  • Truncate long file paths in PR comments for better readability

Full Changelog:
granodigital/report-annotate@v3.1.1...v3.1.2


Configuration

📅 Schedule: Branch creation - "every weekend" in timezone
America/Chicago, Automerge - At any time (no schedule defined).

🚦 Automerge: Enabled.

Rebasing: Whenever PR is behind base branch, or you tick the
rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update
again.


  • If you want to rebase/retry this PR, check
    this box

This PR was generated by Mend Renovate.
View the repository job
log
.

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
- chore(deps): update dependency deno to v2.6.5 (#127)

This PR contains the following updates:

Package Update Change
deno patch 2.6.4
2.6.5

Release Notes

denoland/deno (deno)

v2.6.5

Compare
Source

  • feat(ext/canvas): support gif, webp for createImageBitmap
    (#​31402)
  • feat(ext/web): add performance.clearResourceTimings() and
    setResourceTimingBufferSize()
    (#​31603)
  • feat(node/fs): implement FileHandle.readableWebStream()
    (#​31745)
  • fix(audit): respect --level flag for exit code
    (#​31721)
  • fix(ci): rename trusted-signing-action to artifact-signing-action
    (#​31849)
  • fix(cli/test): close Node HTTP connections before sanitizer check
    (#​31782)
  • fix(ext/fetch): read file urls via vfs
    (#​31814)
  • fix(ext/fetch): support ipv6 destination host with http and socks
    proxy

(#​31683)

  • fix(ext/node): DatabaseSync garbage collection invalidating
    associated
    resources
    (#​31737)
  • fix(ext/node): add tls.setDefaultCACertificates support
    (#​31522)
  • fix(ext/node): emit deprecation warning for SlowBuffer
    (#​31772)
  • fix(ext/node): fix use-after-free in StatementSync JS iterator
    (#​31770)
  • fix(ext/node): use correct block sizes for HMAC algorithms
    (#​31775)
  • fix(ext/web): handle empty TextDecoder input when stream is true
    (#​31691)
  • fix(io): ignore BrokenPipe error in op_print
    (#​31844)
  • fix(node/child_process): ChildProcess constructor without args
    (#​31838)
  • fix(node/child_process): improve arg type validation for execFile and
    fork

(#​31829)

  • fix(node/child_process): reject null bytes in arguments
    (#​31828)
  • fix(node/child_process): use correct syscall name in spawn error
    (#​31836)
  • fix(npm): deduplication pass for initially partially resolved graphs
    (#​31787)
  • fix(runtime): handle options parameter in Deno.errors.*
    constructors

(#​31510)

  • fix(task): preserve signal-based exit codes for child processes
    (#​31789)
  • fix(tsgo): nil deref for exported symbols
    (#​31786)
  • fix(webgpu): correctly expose GPUCompilationInfo &
    GPUCompilationMessage

(#​31780)

  • fix: add more details when panicking getting dir path from deno.json
    (#​31805)
  • fix: allow multiple --allow-all args
    (#​31833)
  • fix: queue dynamic imports for modules with pending TLA
    (#​31670)
  • fix: update config schema for read permission to include ignore
    property

(#​31785)


Configuration

📅 Schedule: Branch creation - "every weekend" in timezone
America/Chicago, Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you
are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update
again.


  • If you want to rebase/retry this PR, check
    this box

This PR was generated by Mend Renovate.
View the repository job
log
.

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
- chore(deps): pin asdf-vm/actions action to b7bcd02 (#126)

This PR contains the following updates:

Package Type Update Change
asdf-vm/actions
action pinDigest b7bcd02

Configuration

📅 Schedule: Branch creation - "every weekend" in timezone
America/Chicago, Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you
are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update
again.


  • If you want to rebase/retry this PR, check
    this box

This PR was generated by Mend Renovate.
View the repository job
log
.

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

@coveralls
Copy link
Copy Markdown

coveralls commented Jan 18, 2026

Pull Request Test Coverage Report for Build 21143968078

Details

  • 5 of 5 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.02%) to 81.616%

Totals Coverage Status
Change from base Build 21096079391: 0.02%
Covered Lines: 1335
Relevant Lines: 1668

💛 - Coveralls

levibostian added a commit to levibostian/decaf-sdk-deno that referenced this pull request Jan 18, 2026
decaf is going to do a rename of the environment variable - from DATA_FILE_PATH to DECAF_COMM_FILE_PATH. learn more: levibostian/decaf#130

this commit prepares for that change by adding support for DECAF_COMM_FILE_PATH while still keeping backwards compatible support until v1.0
@levibostian levibostian changed the title refactor: add support for new user script comms file environment variable DECAF_COMM_FILE_PATH feat: add support for new user script comms file environment variable DECAF_COMM_FILE_PATH Jan 18, 2026
@levibostian levibostian changed the title feat: add support for new user script comms file environment variable DECAF_COMM_FILE_PATH feat: rename the environment variable used to communicate wtih user scripts Jan 18, 2026
Getting error in test mode decaf deployments in PR:
```
> git push '--dry-run'
fatal: could not read Username for 'https://github.com': No such device or address
```

This code was introduced in the previous pull request that I just merged in, which added the dry run. I'm wondering if, for some reason, that's just not working.

I looked at the CI runs, and this command simply did not run in test mode in the previous pull request. So, I have a good feeling that this just isn't going to work.
@levibostian levibostian merged commit 8c5744c into main Jan 19, 2026
11 checks passed
@levibostian levibostian deleted the rename-DATA_FILE_PATH-env-var branch January 19, 2026 16:13
levibostian added a commit to levibostian/decaf-sdk-deno that referenced this pull request Jan 19, 2026
…cripts (#8)

decaf is going to do a rename of the environment variable - from
DATA_FILE_PATH to DECAF_COMM_FILE_PATH. learn more:
levibostian/decaf#130

this commit prepares for that change by adding support for
DECAF_COMM_FILE_PATH while still keeping backwards compatible support
until v1.0
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.

2 participants