Title: Agent config: avoid introducing unused arguments to new functions · Issue #1621 · usethis-python/usethis-python · GitHub
Open Graph Title: Agent config: avoid introducing unused arguments to new functions · Issue #1621 · usethis-python/usethis-python
X Title: Agent config: avoid introducing unused arguments to new functions · Issue #1621 · usethis-python/usethis-python
Description: Let's actually remove the always_run argument - it's always true. And remove pass_filenames - it's always false. And remove require_serial - it's unneeded when pass_filenames is false. Originally posted by @nathanjmcdougall in #1606 (com...
Open Graph Description: Let's actually remove the always_run argument - it's always true. And remove pass_filenames - it's always false. And remove require_serial - it's unneeded when pass_filenames is false. Originally p...
X Description: Let's actually remove the always_run argument - it's always true. And remove pass_filenames - it's always false. And remove require_serial - it's unneeded when pass_filenames is fal...
Opengraph URL: https://github.com/usethis-python/usethis-python/issues/1621
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Agent config: avoid introducing unused arguments to new functions ","articleBody":"Let's actually remove the `always_run` argument - it's always true.\r\nAnd remove `pass_filenames` - it's always false.\r\nAnd remove `require_serial` - it's unneeded when `pass_filenames` is false.\r\n\r\n_Originally posted by @nathanjmcdougall in https://github.com/usethis-python/usethis-python/pull/1606#discussion_r3000808482_\r\n\r\n\r\nThis is the YAGNI principle in action. We should create an agent skill which explains some principles of good function design. One of them is YAGNI for function arguments. We shouldn't pre-emptively introduce arguments unless they are needed at the UI level; or if they are helper/internal functions then we definitely shouldn't introduce arguments which are always passed with the same value/non-default values.","author":{"url":"https://github.com/nathanjmcdougall","@type":"Person","name":"nathanjmcdougall"},"datePublished":"2026-03-27T12:50:49.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/1621/usethis-python/issues/1621"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:dee1d3c0-0306-2a8a-eef9-1673ea74d5a3 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | E4DC:C2F0E:36D4BF:4E4472:6A4C9ECD |
| html-safe-nonce | 95416eda3c027d916fe642d938e16bd335d8bf25c6bd1e0c44485a7841f91d2e |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFNERDOkMyRjBFOjM2RDRCRjo0RTQ0NzI6NkE0QzlFQ0QiLCJ2aXNpdG9yX2lkIjoiMjA0NTkwNzM4ODYzNTk1MjEzIiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | 84d2995f5cb02c189e40eb79e40777c03c370d828c35cf05bb86f30b161a254f |
| hovercard-subject-tag | issue:4152363160 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/usethis-python/usethis-python/1621/issue_layout |
| twitter:image | https://opengraph.githubassets.com/a471ade59a26991aad7bd5c353ac05e1f36a8f47eb30a5c347a8837316fcd8c8/usethis-python/usethis-python/issues/1621 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/a471ade59a26991aad7bd5c353ac05e1f36a8f47eb30a5c347a8837316fcd8c8/usethis-python/usethis-python/issues/1621 |
| og:image:alt | Let's actually remove the always_run argument - it's always true. And remove pass_filenames - it's always false. And remove require_serial - it's unneeded when pass_filenames is false. Originally p... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | nathanjmcdougall |
| hostname | github.com |
| expected-hostname | github.com |
| None | 3d11bb817438277de2a940854450e83a7d32b6aeb5014e9e6b00a6423900251c |
| turbo-cache-control | no-preview |
| go-import | github.com/usethis-python/usethis-python git https://github.com/usethis-python/usethis-python.git |
| octolytics-dimension-user_id | 216362695 |
| octolytics-dimension-user_login | usethis-python |
| octolytics-dimension-repository_id | 842189705 |
| octolytics-dimension-repository_nwo | usethis-python/usethis-python |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 842189705 |
| octolytics-dimension-repository_network_root_nwo | usethis-python/usethis-python |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | ae90d426644ca15e89bacceb72e51f4e9dbf85f7 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width