Skip to content

Conversation

@PEZ
Copy link
Contributor

@PEZ PEZ commented Sep 30, 2025

  • Updating instructions, chatmode and prompt for Clojure interactive programming.
  • Adds a collection for Clojure interactive programming
  • Deletes the add-clojure-memory prompt file, because the use case is covered by the general remember prompt.

Pull Request Checklist

  • I have read and followed the CONTRIBUTING.md guidelines.
  • My contribution adds a new instruction, prompt, or chat mode file in the correct directory.
  • The file follows the required naming convention.
  • The content is clearly structured and follows the example format.
  • I have tested my instructions, prompt, or chat mode with GitHub Copilot.
  • I have run node update-readme.js and verified that README.md is up to date.

Description


Type of Contribution

  • New Collections file
  • Updates an instruction file.
  • Updates a prompt file.
  • Updates a chat mode file.
  • Deletes a prompt file
  • Other (please specify):

Additional Notes


By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.

Copilot AI review requested due to automatic review settings September 30, 2025 21:37
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates and reorganizes Clojure interactive programming resources to improve REPL-first development workflows. The changes consolidate instructions, update prompts with better guidance for interactive programming patterns, and create a new collection to group related resources.

Key changes:

  • Consolidates clojure-memory.instructions.md into a comprehensive clojure.instructions.md file
  • Updates the interactive programming prompt with spelling fixes and clearer guidance
  • Creates a new Clojure Interactive Programming collection grouping related resources
  • Removes the obsolete clojure-add-to-memory prompt (superseded by general remember prompt)

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
prompts/remember-interactive-programming.prompt.md Updates description and content with spelling fix and enhanced guidance
prompts/clojure-add-to-memory.prompt.md Deleted - functionality covered by general remember prompt
instructions/clojure.instructions.md New comprehensive Clojure development instructions consolidating previous memory file
instructions/clojure-memory.instructions.md Deleted - content consolidated into clojure.instructions.md
collections/clojure-interactive-programming.md New collection markdown file for the collection
collections/clojure-interactive-programming.collection.yml New collection definition file
chatmodes/clojure-interactive-programming.chatmode.md Updates with enhanced evaluation guidelines and better formatting
README.prompts.md Removes deleted prompt and updates existing prompt description
README.instructions.md Updates to reference new consolidated Clojure instructions
README.collections.md Adds new Clojure Interactive Programming collection

---

Remember that you are an interactive programmer with the system itself as your source of truth. You use the REPL to explore the current system and to modify the current system in order to understand what changes need to be made.
Remember that you are an interactive programmer with the system itself as your source of thruth. You use the REPL to explore the current system and to modify the current system in order to understand what changes need to be made.
Copy link

Copilot AI Sep 30, 2025

Choose a reason for hiding this comment

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

There's a spelling error: 'thruth' should be 'truth'.

Suggested change
Remember that you are an interactive programmer with the system itself as your source of thruth. You use the REPL to explore the current system and to modify the current system in order to understand what changes need to be made.
Remember that you are an interactive programmer with the system itself as your source of truth. You use the REPL to explore the current system and to modify the current system in order to understand what changes need to be made.

Copilot uses AI. Check for mistakes.
Comment on lines +192 to +204
### Docstrings in `defn`
Docstrings belong immediately after the function name and before the argument vector.

```clojure
(defn my-function
"This function does something."
[arg1 arg2]
;; function body
)
```

- Define functions before they are used—prefer ordering over `declare` except when truly necessary.

Copy link

Copilot AI Sep 30, 2025

Choose a reason for hiding this comment

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

This section about docstrings and function ordering is duplicated at lines 21-32 and 192-203. Consider consolidating this information into a single section to avoid duplication and maintenance issues.

Suggested change
### Docstrings in `defn`
Docstrings belong immediately after the function name and before the argument vector.
```clojure
(defn my-function
"This function does something."
[arg1 arg2]
;; function body
)
```
- Define functions before they are used—prefer ordering over `declare` except when truly necessary.

Copilot uses AI. Check for mistakes.
@aaronpowell aaronpowell merged commit 0319902 into github:main Oct 1, 2025
2 checks passed
@PEZ
Copy link
Contributor Author

PEZ commented Oct 1, 2025

@aaronpowell When putting together the Clojure Collection, I almost included the non-Clojure-related prompts/remember.prompt.md. Because it is super helpful in the Clojure Interactive Programming workflow. The workflow can be super powerful, but there are some mistakes the AI makes that breaks it, and then each time I use the remember prompt it makes Copilot a more awesome interactive programmer.

So, WDYS, is it fine with a generally good prompt in the mix of a collection for a specific workflow?

@PEZ PEZ deleted the update-clojure-instructions branch October 1, 2025 16:27
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