docs: remove outdated group naming note

Allow jump host definitions to include an explicit port in the form user@host:port.
Default to port 22 when no port is provided and update the sample config and docs accordingly.

.memory/project.md

@@ -8,3 +8,6 @@
 - Sample config is stored in `sample-config/` and mirrors current script behavior.
 - Documentation language: English.
 - Current task focus: keep README and memory aligned with the working script and sample configuration.
+- Sample host files currently use numbered names: `01_prod.yaml`, `02_staging.yaml`, `03_dev.yaml`.
+- Known follow-up improvements:
+	- Each group should have default values.

README.md

@@ -5,6 +5,8 @@ A Bash SSH target picker built around `fzf` and YAML configuration.
 ## What it does
 
 - Lists SSH targets from grouped YAML files
+- Uses the `group` value from each YAML file as the displayed group name
+- Allows the YAML file name to differ from the group name
 - Supports global defaults for user, port, and jump hosts
 - Supports per-host overrides
 - Can also browse entries from `~/.ssh/known_hosts`
@@ -45,12 +47,14 @@ ssh:
   default_options: ""
   jump_hosts:
     office: "user@192.168.10.11"
+    office-alt: "user@192.168.10.11:2222"
+
 ```
 
-### `hosts/dev.yaml`
+### `hosts/01_prod.yaml`
 
 ```yaml
-group: dev
+group: prod
 servers:
   - name: api-1
     aliases:
@@ -69,10 +73,10 @@ servers:
     port: 22
 ```
 
-### `hosts/prod.yaml`
+### `hosts/03_dev.yaml`
 
 ```yaml
-group: prod
+group: dev
 servers:
   - name: web-1
     aliases:
@@ -82,6 +86,10 @@ servers:
     jump_host: office
 ```
 
+The file name does not need to match the `group` value. The script reads the `group` from the YAML content and uses the file path internally to load the selected host.
+
+The sample configuration now uses numbered file names such as `01_prod.yaml`, `02_staging.yaml`, and `03_dev.yaml`.
+
 Each host entry can define:
 
 - `name`
@@ -98,9 +106,12 @@ A complete sample configuration is available in `sample-config/`.
 ## Notes
 
 - `jump_host` values are resolved by name from `global.yaml`.
+- Jump hosts support `user@host` and `user@host:port`; if the port is omitted, `22` is used.
 - If a host does not define a user or port, global defaults are used.
 - `default_options` is present in the config but not consumed by the current script.
 - The sample config includes `ssh_options` in one file, but the current script does not consume it yet.
+- Sample host examples are stored as `sample-config/hosts/01_prod.yaml`, `02_staging.yaml`, and `03_dev.yaml`.
+- The `group` key is required for each host config file; the file name is only an internal storage detail.
 - When `-k` is used, entries from `known_hosts` are shown with the selected user or `root` by default.
 - `known_hosts` entries use port `22`.
 - The script uses strict Bash mode (`set -euo pipefail`).

mtm-ssh-menu

@@ -3,7 +3,7 @@ set -euo pipefail
 
 CONFIG_DIR="$HOME/.config/mtm-ssh-menu"
 USE_KNOWN_HOSTS=0
-USE_KNOWN_HOSTS_USER="root"
+USE_KNOWN_HOSTS_USER=$(whoami)
 
 usage() {
 	cat <<'EOF'
@@ -34,7 +34,6 @@ parse_args() {
 			--known-hosts|-k)
 				USE_KNOWN_HOSTS=1
 				if [ $# -eq 1 ]; then
-					USE_KNOWN_HOSTS_USER="root"
 					shift
 				else
 					USE_KNOWN_HOSTS_USER="$2"
@@ -95,12 +94,13 @@ load_config() {
 	for file in "$HOSTS_DIR"/*.yaml; do
 		local FILE_CONTENT GROUP_NAME GROUP_SERVERS INDEX
 		FILE_CONTENT="$(<"$file")"
-		GROUP_NAME=$(basename "$file")
+		GROUP_NAME=$(yq -r '.group // empty' <<<"$FILE_CONTENT")
-		GROUP_NAME=${GROUP_NAME%.*}
+		if [ "$GROUP_NAME" = "" ]; then
-		GROUP_NAME=$(yq -r '.group // "'"$GROUP_NAME"'"' <<<"$FILE_CONTENT")
+			printf 'Error: missing group name in config file: %s\n' "$file" >&2
+			exit 1
+		fi
 		GROUP_SERVERS=$(yq -r '.servers // []' <<<"$FILE_CONTENT")
 		INDEX=0
-		
 		for ((i=0; i<$(jq -r '. | length' <<<"$GROUP_SERVERS"); i++)); do
 			local SSH_SERVER_NAME SSH_SERVER_ALIASES ALIASES SSH_SERVER_HOST SSH_SERVER_PORT SSH_SERVER_USER
 			SSH_SERVER_NAME=$(jq -r '.'"[$i]"'.name // ""' <<<"$GROUP_SERVERS")
@@ -108,6 +108,7 @@ load_config() {
 			SSH_SERVER_HOST=$(jq -r '.'"[$i]"'.host // ""' <<<"$GROUP_SERVERS")
 			SSH_SERVER_PORT=$(jq -r '.'"[$i]"'.port // "'"$DEFAULT_SSH_PORT"'"' <<<"$GROUP_SERVERS")
 			SSH_SERVER_USER=$(jq -r '.'"[$i]"'.user // "'"$DEFAULT_SSH_USER"'"' <<<"$GROUP_SERVERS")
+			ALIASES=""
 			if [ "$SSH_SERVER_ALIASES" != "" ]; then
 				# shellcheck disable=SC2027
 				ALIASES="("$(echo "$SSH_SERVER_ALIASES" | jq -r 'join(", ")')")"
@@ -115,10 +116,9 @@ load_config() {
 			if [ "$SERVERS" != "" ]; then
 				SERVERS+="\n"
 			fi
-			SERVERS+="${GROUP_NAME} | ${INDEX} | ${SSH_SERVER_HOST} | ${SSH_SERVER_NAME} ${ALIASES} | ${SSH_SERVER_USER} | ${SSH_SERVER_PORT}"
+			SERVERS+="${GROUP_NAME} | ${file} | ${INDEX} | ${SSH_SERVER_HOST} | ${SSH_SERVER_NAME} ${ALIASES} | ${SSH_SERVER_USER} | ${SSH_SERVER_PORT}"
 			INDEX=$((INDEX+1))
 		done
-
 	done
 	shopt -u nullglob
 }
@@ -130,21 +130,22 @@ load_known_hosts() {
 		if [ "$SERVERS" != "" ]; then
 			SERVERS+="\n"
 		fi
-		SERVERS+="${GROUP_NAME} | ${host} | ${host} | ${host} | ${USE_KNOWN_HOSTS_USER} | 22"
+		SERVERS+="${GROUP_NAME} | known_hosts | ${host} | ${host} | ${USE_KNOWN_HOSTS_USER} | 22"
 	done
 }
 popup_menu() {
 	local SERVER
-	SERVER=$(echo -e "${SERVERS}" | column -t -s "|" -o "|" | fzf -e --layout=reverse --with-nth=1,3,4,5,6 --delimiter="|")
+	SERVER=$(echo -e "${SERVERS}" | column -t -s "|" -o "|" | fzf -e --layout=reverse --with-nth=1,4,5,6,7 --delimiter="|")
-	local GROUP_NAME HOST_INDEX
+	local GROUP_NAME GROUP_SOURCE HOST_INDEX
 	GROUP_NAME=$(echo "$SERVER" | awk -F '|' '{print $1}' | xargs)
-	HOST_INDEX=$(echo "$SERVER" | awk -F '|' '{print $2}' | xargs)
+	GROUP_SOURCE=$(echo "$SERVER" | awk -F '|' '{print $2}' | xargs)
-	ssh_connect "$GROUP_NAME" "$HOST_INDEX"
+	HOST_INDEX=$(echo "$SERVER" | awk -F '|' '{print $3}' | xargs)
+	ssh_connect "$GROUP_SOURCE" "$HOST_INDEX"
 }
 ssh_connect() {
 	if [[ "$HOST_INDEX" != *"."* ]]; then
 		local SERVER SSH_SERVER_USER SSH_SERVER_HOST SSH_SERVER_PORT SSH_SERVER_OPTIONS SSH_JUMP_HOST
-		SERVER=$(cat "$HOSTS_DIR/$1.yaml" | yq -r '.servers'"[$2]")
+		SERVER=$(yq -r '.servers['"$2"']' "$1")
 		SSH_SERVER_USER=$(jq -r '.user // "'"$DEFAULT_SSH_USER"'"' <<<"$SERVER")
 		SSH_SERVER_HOST=$(jq -r '.host // ""' <<<"$SERVER")
 		SSH_SERVER_PORT=$(jq -r '.port // "'"$DEFAULT_SSH_PORT"'"' <<<"$SERVER")
@@ -153,7 +154,14 @@ ssh_connect() {
 		SSH_JUMP_HOST=$(jq -r '.jump_host // ""' <<<"$SERVER")
 		if [ "$SSH_JUMP_HOST" != "" ]; then
 			SSH_JUMP_HOST=$(jq -r '.'"$SSH_JUMP_HOST"' // ""' <<<"${SSH_JUMP_HOSTS}")
-			SSH_JUMP_HOST="-t $SSH_JUMP_HOST ssh -p $SSH_SERVER_PORT"
+			SSH_JUMP_HOST_USER=$(cut -d'@' -f1 <<<"$SSH_JUMP_HOST")
+			SSH_JUMP_HOST_TARGET=$(cut -d'@' -f2- <<<"$SSH_JUMP_HOST")
+			SSH_JUMP_HOST_TARGET_HOST=$(cut -d':' -f1 <<<"$SSH_JUMP_HOST_TARGET")
+			SSH_JUMP_HOST_TARGET_PORT=$(cut -s -d':' -f2 <<<"$SSH_JUMP_HOST_TARGET")
+			if [ "$SSH_JUMP_HOST_TARGET_PORT" = "" ]; then
+				SSH_JUMP_HOST_TARGET_PORT="22"
+			fi
+			SSH_JUMP_HOST="-t ${SSH_JUMP_HOST_USER}@${SSH_JUMP_HOST_TARGET_HOST} ssh -p ${SSH_JUMP_HOST_TARGET_PORT}"
 			SSH_SERVER_OPTIONS=""
 		fi
 
@@ -165,10 +173,10 @@ ssh_connect() {
 }
 
 main() {
-	load_config
 	if [ "$USE_KNOWN_HOSTS" == 1 ]; then
 		load_known_hosts
 	fi
+	load_config
 	popup_menu
 }
 

sample-config/global.yaml

@@ -4,3 +4,5 @@ ssh:
   default_options: ""
   jump_hosts:
     office: "user@192.168.10.11"
+    dev: "user@192.168.10.11:2222"
+    backup: "admin@10.0.0.5:2200"