Live data from Hacker News

AI-Generated GitHub Copilot “Autofix” Allowed Compromise of Snowflake's Jira

wiz.io

21–30 of 179 posts

Re: AI-Generated GitHub Copilot “Autofix” Allowed Compromise of Snowflake's Jira

#21
post #8
post #4

Earlier quoted context omitted.

You can't rely on people spotting the significance of such changes

Tests would have caught it = https://github.com/rhysd/actionlint injection check

also been a huge fan of zizmor (https://github.com/zizmorcore/zizmor) lately, basically: "am I going to footgun myself?"

Re: AI-Generated GitHub Copilot “Autofix” Allowed Compromise of Snowflake's Jira

#22
post #19

YAML is a nightmare fuel spec. In its quest to make markup "human readable", it has created countless footguns. I honestly prefer XML at this point.

It’s find for actions and workflows as long as you do no interpolation and logic.

Better move as much of that as possible into your own scripts. And your scripts can be portable between forges, and even run locally!

Re: AI-Generated GitHub Copilot “Autofix” Allowed Compromise of Snowflake's Jira

#24
post #22
post #19

YAML is a nightmare fuel spec. In its quest to make markup "human readable", it has created countless footguns. I honestly prefer XML at this point.

It’s find for actions and workflows as long as you do no interpolation and logic. Better move as much of that as possible into your own scripts. And your scripts can be portable between forges, and even run locally!

Assigning an env var to an empty variable (env: { myvar: ${{unsetfoo}} }) should trigger an error, not silently pass an empty string.

Re: AI-Generated GitHub Copilot “Autofix” Allowed Compromise of Snowflake's Jira

#26
post #19

YAML is a nightmare fuel spec. In its quest to make markup "human readable", it has created countless footguns. I honestly prefer XML at this point.

In a similar vein, JSON's lack of comments makes me marvel at how consistently JavaScript seems to choose the worse option. I'm oh so glad it found its way into config files

Re: AI-Generated GitHub Copilot “Autofix” Allowed Compromise of Snowflake's Jira

#27
post #7
post #3

Peer review of changes is still important.

Multi-model cross-review is important

I'm all for using a council of LLMs, I wrote a tool for it https://github.com/joelio/owl - but you still need to read through PRs yourself, at the very least.

Re: AI-Generated GitHub Copilot “Autofix” Allowed Compromise of Snowflake's Jira

#28
post #24
post #22

Earlier quoted context omitted.

It’s find for actions and workflows as long as you do no interpolation and logic. Better move as much of that as possible into your own scripts. And your scripts can be portable between forges, and even run locally!

Assigning an env var to an empty variable (env: { myvar: ${{unsetfoo}} }) should trigger an error, not silently pass an empty string.

What does that have to do with YAML?

Re: AI-Generated GitHub Copilot “Autofix” Allowed Compromise of Snowflake's Jira

#30
post #19

YAML is a nightmare fuel spec. In its quest to make markup "human readable", it has created countless footguns. I honestly prefer XML at this point.

And thus procone spoketh the truth.

In accordance with the prophecy.
Post reply on HN