Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
45 changes: 33 additions & 12 deletions .github/ISSUE_TEMPLATE/bug.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,12 +8,12 @@ body:
- type: textarea

attributes:
label: Description

description: >-
Describe the issue in detail. Consider including a minimal reproducible
example, logs, or screenshots.

label: Description

placeholder: Type your description here…

validations:
Expand All @@ -22,12 +22,12 @@ body:
- type: textarea

attributes:
label: flake.lock

description: >-
Copy-paste the `flake.lock` file. If flakes are not used, remove the
prefilled content and provide the locked input versions.

label: flake.lock

value: |-
<details>
<summary><code>flake.lock</code></summary>
Expand All @@ -44,8 +44,8 @@ body:
- type: dropdown

attributes:
description: How is Stylix installed?
label: Installation Method
description: How is Stylix installed?

options:
- NixOS
Expand All @@ -60,16 +60,16 @@ body:
- type: textarea

attributes:
description: "`nix-info --markdown` output."
label: System Information
description: "`nix-info --markdown` output."

placeholder: |-
- system:
- host os:
- multi-user?:
- sandbox:
- version:
- nixpkgs:
- system:
- host os:
- multi-user?:
- sandbox:
- version:
- nixpkgs:

validations:
required: true
Expand Down Expand Up @@ -104,6 +104,27 @@ body:

required: true

- type: dropdown

attributes:
label: LLM-involvement Category

description: >-
Disclose the most most severe LLM-involvement category that applies to
this issue.

options:
- >-
`llm: 0`: Manual: The absence of LLMs would result in equal results
and quality

- "`llm: 1`: Syntactic: Includes non-negligible LLM content"
- "`llm: 2`: Semantic: LLM involved in design decisions or core logic"
- "`llm: 3`: Unaudited: LLM output with little to no human verification"

validations:
required: true

- type: textarea

attributes:
Expand Down
22 changes: 22 additions & 0 deletions .github/ISSUE_TEMPLATE/change_option.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,28 @@ body:

required: true

- type: dropdown

attributes:
label: LLM-involvement Category

description: >-
Disclose the most most severe LLM-involvement category that applies to
this issue.

options:
- >-
`llm: 0`: Manual: The absence of LLMs would result in equal results
and quality

- "`llm: 1`: Syntactic: Includes non-negligible LLM content"
- "`llm: 2`: Semantic: LLM involved in design decisions or core logic"
- "`llm: 3`: Unaudited: LLM output with little to no human verification"

validations:
required: true


- type: textarea

attributes:
Expand Down
22 changes: 22 additions & 0 deletions .github/ISSUE_TEMPLATE/change_target.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,28 @@ body:

required: true

- type: dropdown

attributes:
label: LLM-involvement Category

description: >-
Disclose the most most severe LLM-involvement category that applies to
this issue.

options:
- >-
`llm: 0`: Manual: The absence of LLMs would result in equal results
and quality

- "`llm: 1`: Syntactic: Includes non-negligible LLM content"
- "`llm: 2`: Semantic: LLM involved in design decisions or core logic"
- "`llm: 3`: Unaudited: LLM output with little to no human verification"

validations:
required: true


- type: textarea

attributes:
Expand Down
22 changes: 22 additions & 0 deletions .github/ISSUE_TEMPLATE/new_option.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,28 @@ body:

required: true

- type: dropdown

attributes:
label: LLM-involvement Category

description: >-
Disclose the most most severe LLM-involvement category that applies to
this issue.

options:
- >-
`llm: 0`: Manual: The absence of LLMs would result in equal results
and quality

- "`llm: 1`: Syntactic: Includes non-negligible LLM content"
- "`llm: 2`: Semantic: LLM involved in design decisions or core logic"
- "`llm: 3`: Unaudited: LLM output with little to no human verification"

validations:
required: true


- type: textarea

attributes:
Expand Down
21 changes: 21 additions & 0 deletions .github/ISSUE_TEMPLATE/new_target.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,3 +33,24 @@ body:
issue](https://github.com/nix-community/stylix/issues?q=is%3Aissue)

required: true

- type: dropdown

attributes:
label: LLM-involvement Category

description: >-
Disclose the most most severe LLM-involvement category that applies to
this issue.

options:
- >-
`llm: 0`: Manual: The absence of LLMs would result in equal results
and quality

- "`llm: 1`: Syntactic: Includes non-negligible LLM content"
- "`llm: 2`: Semantic: LLM involved in design decisions or core logic"
- "`llm: 3`: Unaudited: LLM output with little to no human verification"

validations:
required: true
5 changes: 5 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,11 @@ Unless otherwise specified, the following checkboxes are not mandatory, but
drastically accelerate the reviewing and merging process of this PR.
-->
- [ ] <!-- MANDATORY --> I certify that I have the right to submit this contribution under the [MIT license](https://github.com/nix-community/stylix/blob/master/LICENSE)
- [ ] <!-- MANDATORY --> I have disclosed the single most severe LLM-involvement category that applies to this contribution:
- [ ] `llm: 0`: Manual: The absence of LLMs would result in equal results and quality
- [ ] `llm: 1`: Syntactic: Includes non-negligible LLM content
Comment on lines +13 to +14

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

llm: 0 is very open to interpretation, where as the other three are significantly more clear. I would prefer having llm: 0 be non-existent LLM usage and llm: 1 be any LLM usage,

Suggested change
- [ ] `llm: 0`: Manual: The absence of LLMs would result in equal results and quality
- [ ] `llm: 1`: Syntactic: Includes non-negligible LLM content
- [ ] `llm: 0`: Manual: LLMs were absent in the creation of this PR
- [ ] `llm: 1`: Syntactic: Includes LLM generated content

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

llm: 0 is very open to interpretation, where as the other three are significantly more clear. I would prefer having llm: 0 be non-existent LLM usage

It intentionally has a faithfull loophole inspired by:

These rules apply only to outside contributions to Ghostty. Maintainers
are exempt from these rules and may use AI tools at their discretion;
they've proven themselves trustworthy to apply good judgment.

-- github:ghostty-org/ghostty, /AI_POLICY.md

I suggest initially going with this, allowing competent contributors to faithfully disclose their LLM dependency because it seems well-regarded Nix maintainers use LLMs for minor things. Regardless of the label description, it is suspect to dishonesty that I suggest ignoring for now:

Considering the inherent LLM discrimination introduced by the 13.llm: n labels, malicious or dishonest contributors might intentionally miscategorize their contributions towards 13.llm: 0 to potentially avoid being ignored by LLM skeptics. Unless it becomes obvious that a non-negligible amount of contributions are intentionally miscategorized, I suggest faithfully assuming contributors to be honest, before designing a LLM ban procedure.

-- NixOS/nixpkgs#410741 (comment)

and llm: 1 be any LLM usage

This also intentionally has a smaller loophole, allowing trivial use cases like LLMs fixing trivial syntax issues or suggesting the most idiomatic function to use. I suppose what this really entails will be clear once we see how people interpret this.

I see your point of more explicitly disclosing LLM contributions, but I suggest going with this for now and later determining how much people abuse these faithfull definitions.

- [ ] `llm: 2`: Semantic: LLM involved in design decisions or core logic
- [ ] `llm: 3`: Unaudited: LLM output with little to no human verification

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We should make it clear that is level of LLM involvement is not welcome in Stylix, as was agreed on in #2178.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

We should make it clear that is level of LLM involvement is not welcome in Stylix, as was agreed on in #2178.

IMHO, the categorization is already fairly discriminatory towards LLM contributions, making it somewhat obvious that unaudited contributions are unwelcome. These are also rude submissions, but this is irrelevant to the LLM categorization.

I wonder whether we will even receive submissions categorized as unaudited. Keep in mind that you are free to ignore such submissions or only very superficially interact with them.

- [ ] Commit messages adhere to [Stylix commit conventions](https://nix-community.github.io/stylix/commit_convention.html)
- [ ] Theming changes adhere to the [Stylix style guide](https://nix-community.github.io/stylix/styling.html)
- [ ] Changes have been [tested locally](https://nix-community.github.io/stylix/modules.html#development-setup)
Expand Down
Loading