Refactor and Fix ECS Agent Deployment (#152)

* Agent Implementation Using Pydantic AI (#149)

* Move old stuff to a legacy folder and setup pre-commit for rewrite

* Agent rewrite

* Update cloud watch config

* Update prompt and use remote github mcp server

* Add cloud watch tool log

* Use remote github mcp for now

* Update agent

* Update dev doc

* Make service name a required parameter

* Update cloud watch tool to search directly from log event

* Cli

* Readme

* Update agents.md

* ECS deployment

* Simplify by removing server for now

* Add logo

* Update text colour

* Update

* Refactoring

* Refactoring

Author: osw282

.gitignore

@@ -1,6 +1,6 @@
 # Byte-compiled / optimized / DLL files
 __pycache__/
-*.py[cod]
+*.py[codz]
 *$py.class
 
 # C extensions
@@ -27,8 +27,8 @@ share/python-wheels/
 MANIFEST
 
 # PyInstaller
-#  Usually these files are written by a python script from a template
-#  before PyInstaller builds the exe, so as to inject date/other infos into it.
+#   Usually these files are written by a python script from a template
+#   before PyInstaller builds the exe, so as to inject date/other infos into it.
 *.manifest
 *.spec
 
@@ -46,7 +46,7 @@ htmlcov/
 nosetests.xml
 coverage.xml
 *.cover
-*.py,cover
+*.py.cover
 .hypothesis/
 .pytest_cache/
 cover/
@@ -92,22 +92,37 @@ ipython_config.py
 #   However, in case of collaboration, if having platform-specific dependencies or dependencies
 #   having no cross-platform support, pipenv may install dependencies that don't work, or not
 #   install all needed dependencies.
-#Pipfile.lock
+# Pipfile.lock
+
+# UV
+#   Similar to Pipfile.lock, it is generally recommended to include uv.lock in version control.
+#   This is especially recommended for binary packages to ensure reproducibility, and is more
+#   commonly ignored for libraries.
+# uv.lock
 
 # poetry
 #   Similar to Pipfile.lock, it is generally recommended to include poetry.lock in version control.
 #   This is especially recommended for binary packages to ensure reproducibility, and is more
 #   commonly ignored for libraries.
 #   https://python-poetry.org/docs/basic-usage/#commit-your-poetrylock-file-to-version-control
-#poetry.lock
+# poetry.lock
+# poetry.toml
 
 # pdm
 #   Similar to Pipfile.lock, it is generally recommended to include pdm.lock in version control.
-#pdm.lock
-#   pdm stores project-wide configurations in .pdm.toml, but it is recommended to not include it
-#   in version control.
-#   https://pdm.fming.dev/#use-with-ide
-.pdm.toml
+#   pdm recommends including project-wide configuration in pdm.toml, but excluding .pdm-python.
+#   https://pdm-project.org/en/latest/usage/project/#working-with-version-control
+# pdm.lock
+# pdm.toml
+.pdm-python
+.pdm-build/
+
+# pixi
+#   Similar to Pipfile.lock, it is generally recommended to include pixi.lock in version control.
+# pixi.lock
+#   Pixi creates a virtual environment in the .pixi directory, just like venv module creates one
+#   in the .venv directory. It is recommended not to include this directory in version control.
+.pixi
 
 # PEP 582; used by e.g. github.com/David-OConnor/pyflow and github.com/pdm-project/pdm
 __pypackages__/
@@ -116,11 +131,25 @@ __pypackages__/
 celerybeat-schedule
 celerybeat.pid
 
+# Redis
+*.rdb
+*.aof
+*.pid
+
+# RabbitMQ
+mnesia/
+rabbitmq/
+rabbitmq-data/
+
+# ActiveMQ
+activemq-data/
+
 # SageMath parsed files
 *.sage.py
 
 # Environments
 .env
+.envrc
 .venv
 env/
 venv/
@@ -153,103 +182,38 @@ dmypy.json
 cython_debug/
 
 # PyCharm
-#  JetBrains specific template is maintained in a separate JetBrains.gitignore that can
-#  be found at https://github.com/github/gitignore/blob/main/Global/JetBrains.gitignore
-#  and can be added to the global gitignore or merged into this file.  For a more nuclear
-#  option (not recommended) you can uncomment the following to ignore the entire idea folder.
-.idea/
-
-### macOS ###
-# General
+#   JetBrains specific template is maintained in a separate JetBrains.gitignore that can
+#   be found at https://github.com/github/gitignore/blob/main/Global/JetBrains.gitignore
+#   and can be added to the global gitignore or merged into this file.  For a more nuclear
+#   option (not recommended) you can uncomment the following to ignore the entire idea folder.
+# .idea/
+
+# Abstra
+#   Abstra is an AI-powered process automation framework.
+#   Ignore directories containing user credentials, local state, and settings.
+#   Learn more at https://abstra.io/docs
+.abstra/
+
+# Visual Studio Code
+#   Visual Studio Code specific template is maintained in a separate VisualStudioCode.gitignore 
+#   that can be found at https://github.com/github/gitignore/blob/main/Global/VisualStudioCode.gitignore
+#   and can be added to the global gitignore or merged into this file. However, if you prefer, 
+#   you could uncomment the following to ignore the entire vscode folder
+# .vscode/
+
+# Ruff stuff:
+.ruff_cache/
+
+# PyPI configuration file
+.pypirc
+
+# Marimo
+marimo/_static/
+marimo/_lsp/
+__marimo__/
+
+# Streamlit
+.streamlit/secrets.toml
+
+# .DS_Store
 .DS_Store
-.AppleDouble
-.LSOverride
-
-# Icon must end with two \r
-Icon
-
-
-# Thumbnails
-._*
-
-# Files that might appear in the root of a volume
-.DocumentRevisions-V100
-.fseventsd
-.Spotlight-V100
-.TemporaryItems
-.Trashes
-.VolumeIcon.icns
-.com.apple.timemachine.donotpresent
-
-# Directories potentially created on remote AFP share
-.AppleDB
-.AppleDesktop
-Network Trash Folder
-Temporary Items
-.apdisk
-
-### macOS Patch ###
-# iCloud generated files
-*.icloud
-
-### Linux ###
-*~
-
-# temporary files which can be created if a process still has a handle open of a deleted file
-.fuse_hidden*
-
-# KDE directory preferences
-.directory
-
-# Linux trash folder which might appear on any partition or disk
-.Trash-*
-
-# .nfs files are created when an open file is removed but is still being accessed
-.nfs*
-
-### Windows ###
-# Windows thumbnail cache files
-Thumbs.db
-Thumbs.db:encryptable
-ehthumbs.db
-ehthumbs_vista.db
-
-# Dump file
-*.stackdump
-
-# Folder config file
-[Dd]esktop.ini
-
-# Recycle Bin used on file shares
-$RECYCLE.BIN/
-
-# Windows Installer files
-*.cab
-*.msi
-*.msix
-*.msm
-*.msp
-
-# Windows shortcuts
-*.lnk
-
-*node_modules
-
-# Terraform
-.terraform
-*.tfstate
-*.tfstate.*
-
-# Ignore Helm dependencies
-charts/*/Chart.lock
-charts/*/tmpcharts/
-
-# Helm-generated files
-*.tgz
-Chart.lock
-tmpcharts/
-
-# Ignore Helm release artifacts
-.release-name/
-
-*values-secrets.yaml

.pre-commit-config.yaml

@@ -2,7 +2,7 @@ fail_fast: false
 
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.5.0
+    rev: v6.0.0
     hooks:
       - id: check-toml
       - id: check-yaml
@@ -18,18 +18,16 @@ repos:
       - id: check-case-conflict
       - id: requirements-txt-fixer
 
-  - repo: https://github.com/psf/black
-    rev: 23.10.1
-    hooks:
-      - id: black
-        args: [--config=pyproject.toml]
-
-  - repo: https://github.com/charliermarsh/ruff-pre-commit
+  - repo: https://github.com/astral-sh/ruff-pre-commit
     # Ruff version.
-    rev: "v0.1.2"
+    rev: v0.14.14
     hooks:
-      - id: ruff
-        args: [--fix, --exit-non-zero-on-fix, "--config=pyproject.toml"] # enable autofix
+      # Run the linter.
+      - id: ruff-check
+        args: [--fix, --config=ruff.toml]
+      # Run the formatter.
+      - id: ruff-format
+        args: [--config=ruff.toml]
 
   - repo: local
     hooks:
@@ -38,22 +36,21 @@ repos:
         entry: mypy
         language: system
         types: [python]
-        args: ["--config-file=pyproject.toml", "--ignore-missing-imports"]
-        exclude: '.*__init__\.py|tests'
+        args: ["--config-file=mypy.ini"]
+        exclude: ^legacy/
 
   - repo: https://github.com/crate-ci/typos
-    rev: v1.32.0
+    rev: v1.42.1
     hooks:
       - id: typos
-        args: [--config=pyproject.toml]
+        args: [--config=typos.toml]
         pass_filenames: false
 
   - repo: https://github.com/PyCQA/bandit
-    rev: 1.7.8
+    rev: 1.9.3
     hooks:
       - id: bandit
-        args: ["--config=pyproject.toml"]
-        additional_dependencies: ["bandit[toml]"]
+        args: ["-c", "bandit.yaml"]
 
   - repo: local
     hooks:

.typos.toml

@@ -0,0 +1,35 @@
+# AGENTS.md
+
+## Do
+- Always use Context7 MCP when I need library/API documentation, code generation, setup or configuration steps without me having to explicitly ask.
+- Keep code simple, with a strong focus on readability and maintainability.
+- Use UK English.
+- No em dashes in comments and documentations.
+
+## Functions
+- Keep each function concise, easy to read, and clearly named.
+- Avoid functions that handle multiple responsibilities. Break them into smaller units unless there is a sensible trade off.
+- Prioritise on readability, maintainability, reusability, and testability.
+
+## Docstrings
+- Keep module-level and script top-level docstrings to a single line.
+- Use Google-style docstrings.
+- Do not include types for arguments.
+- Keep docstrings concise and only include what is necessary to help readers understand the function or class.
+
+### Docstrings Example
+```
+def function_with_pep484_type_annotations(param1: int, param2: str) -> bool:
+    """Example function with PEP 484 type annotations.
+
+    Important note.
+
+    Args:
+        param1: The first parameter.
+        param2: The second parameter.
+
+    Returns:
+        The return value. True for success, False otherwise.
+
+    """
+```